hhr293 commented on PR #12756: URL: https://github.com/apache/gluten/pull/12756#issuecomment-5438347501
> Hi, @hhr293 . After discussing with @zhouyuan and @LuciferYang , I think it’s fine to continue with this change in Gluten, especially to better support older Spark versions. Please consider validating it with upstream Spark CI first, since optimizer changes usually get more comprehensive coverage there. Even if it’s not eventually merged upstream, confirming that it works correctly in the general Spark context should be sufficient for us to move forward in Gluten. Thanks for your great work again. Thanks for the guidance and for discussing this with @zhouyuan and @LuciferYang. I’ll prepare a corresponding Spark-side change and validate it with upstream Spark CI, then report the results back here. Thanks again. -- 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]
