gaborgsomogyi edited a comment on pull request #28623: URL: https://github.com/apache/spark/pull/28623#issuecomment-667014944
I'm just revisiting this. At the moment working on SPARK-32032 where `AdminClient` is planned to use on driver side which makes `group.id` completely useless in the Kafka connector. If it's merged then we can go to this direction but not with the current implementation. I think we shouldn't remove `group.id` from the map but just remove code references which are using it (for instance in `KafkaDataConsumer`) + documentation of course. ---------------------------------------------------------------- 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]
