ArnavBalyan commented on PR #9405: URL: https://github.com/apache/incubator-gluten/pull/9405#issuecomment-2826384442
> @ArnavBalyan > > > was not sure if there was some reason to enable only for 3.5, > > It was for saving CI resources. So we just covered the latest version. > > If there aren't other specific reasons, I think it's fine to keep it in 3.5 for now. Given the optimizer code is 99% similar among different Spark versions. We can tolerate a little coverage loss to save some Ci resource. Yes makes sense, interestingly there is a failure in spark 33 RAS test. I'll take a look and try to fix, makes me think if it would be good to have it. Let me close this for now, can revisit in future if needed? @zhztheplayer -- 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]
