urosstan-db commented on PR #53456: URL: https://github.com/apache/spark/pull/53456#issuecomment-3647527854
Can you update description with additional reason for change needed: - Changed return value from Boolean to Option[Boolean] for newly introduced methods (from Spark 4.1.0) to be able to know whether dialect overrides method, since currently we don't know whether `true` means method is overriden and returns `true` or that is default implementation. For both methods, it does not make sense to have default true/false. -- 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]
