gaborgsomogyi commented on pull request #29729: URL: https://github.com/apache/spark/pull/29729#issuecomment-716492068
@xuanyuanking Thansk for your efforts making this PR better! Let me react your suggestions: > Should we add some guides of the alternative method for the original usage of group.id based authorization? Maybe link the Kafka doc is enough. The `group.id` based authorization uses Kafka ACLs and when one need to switch to topic based then it's the same feature but another field need to be used. Since the feature is the same I don't feel the need. > IMO, It would be great to link all the investigations/tests in the PR description. Updated the doc which is linked. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
