ryan-johnson-databricks commented on PR #37573:
URL: https://github.com/apache/spark/pull/37573#issuecomment-1223300698

   > Since this is a source level incompatibility (as per @JoshRosen's 
analysis), and given this has been around for a while as a `DeveloperApi`, I 
would mark it as deprecated - and relook at removing it in the next major 
release. We can ofcourse cleanup our own use
   
   We can't "just" mark it as deprecated and also clean up our own call sites, 
because this is an ambiguous overload. If we mark the polymorphic function 
overload as deprecated, that just forces everyone to either ignore the warning, 
or to create an actual listener object until we get around to removing the 
deprecated overload. 
   
   Neither of those seems to provide much benefit?


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

To unsubscribe, e-mail: [email protected]

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