HeartSaVioR commented on pull request #29729:
URL: https://github.com/apache/spark/pull/29729#issuecomment-699721072


   @zsxwing @gaborgsomogyi 
   
   Let's talk about the remaining tasks to move this forward. I see the change 
of security requirement is probably a breaking change, but shouldn't be a 
blocker (as it's required to fix a real-world issue).
   
   Which would be the best way to notice the change to the end users? I'd 
propose to add the change on the "SS migration note", and optionally describe 
the security section on "SQL Kafka integration". Adding it to the release note 
would be also great as well.
   
   I might probably be missing something, or trying to be too exhaustive. Open 
to any suggestion. Thanks!


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

Reply via email to