zhztheplayer commented on PR #9312: URL: https://github.com/apache/incubator-gluten/pull/9312#issuecomment-2801778744
> If we need to load to GlutenSessionExtensions by default and we've provided a dynamic config to disable it We load `GlutenSessionExtensions` only when `spark.plugins=org.apache.gluten.GlutenPlugin`. We have a dynamic config `spark.gluten.enabled` to disable Gluten but the option can take effect even after the extensions are injected. So it's implemented in the different way. The PR is just to address a corner case that user accidentally specifies `spark.sql.extensions=org.apache.gluten.extension.GlutenSessionExtensions` manually which is not recommended. We'd make Gluten work normally in that case. -- 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]
