[GitHub] [kafka] cmccabe edited a comment on pull request #8639: KAFKA-9962: Make the auth operations fields ignorable

[GitBox <gi...@apache.org>]

cmccabe edited a comment 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.
   
   > [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:
users@infra.apache.org