squakez commented on code in PR #3912:
URL: https://github.com/apache/camel-k/pull/3912#discussion_r1054093419


##########
pkg/apis/camel/v1/maven_types.go:
##########
@@ -96,4 +96,9 @@ type Server struct {
        Configuration Properties `xml:"configuration,omitempty" 
json:"configuration,omitempty"`
 }
 
-type Properties map[string]string

Review Comment:
   This change here looks to be breaking compatibility in the API. In order to 
be performed we should either have a backward compatible field or deprecate the 
previous one and make it coexist for a couple of versions with the newer.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to