snehashisp commented on code in PR #17988:
URL: https://github.com/apache/kafka/pull/17988#discussion_r1996134289
##########
connect/runtime/src/main/java/org/apache/kafka/connect/runtime/rest/entities/ConnectorStateInfo.java:
##########
@@ -87,14 +90,26 @@ public String workerId() {
public String trace() {
return trace;
}
+
+ @JsonProperty
+ @JsonInclude(JsonInclude.Include.NON_EMPTY)
Review Comment:
We can keep this consistent and add the custom filter hare too. Wdyt of just
showing the version (even if its null or undefined) with multi-versioning
support. People who are unaware of the internals may be confused as to why its
not showing up at all if the version is not set.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
