cmccabe commented on pull request #8639:
URL: https://github.com/apache/kafka/pull/8639#issuecomment-637823813


   **The KIP clearly specifies that `describeCluster` will throw a UVE in this 
case:
   
   bq. When the AdminClient is talking to a broker which does not support 
KIP-430, it will fill in either null or UnsupportedVersionException for the 
returned ACL operations fields in objects.  For example, 
ConsumerGroupDescription#authorizedOperations will be null if the broker did 
not supply this information.  DescribeClusterResult#authorizedOperations will 
throw an UnsupportedVersionException if the broker did not supply this 
information.
   
   The discussion above was about `describeConsumerGroups`, but the PR seems to 
be changing both.
   
   bq. [UVEs are] not fine grain enough for all cases. There are already a few 
reasons why the call may throw an UnsupportedVersionException.
   
   In the future, KIP-584 will make new features (that can't be faked by old 
brokers) more discoverable to clients.  In the meantime, KSQL can have a 
fallback loop where they turn this thing off when they get a UVE and retry.


----------------------------------------------------------------
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.

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


Reply via email to