pan3793 commented on PR #46521: URL: https://github.com/apache/spark/pull/46521#issuecomment-2707065925
> > For this one PR, I believe we need a verification for different HMS versions to make it sure. > > that's a valid concern, since Spark CI only covers embedded HMS client case, let me test it with the real setup. I just found I forgot this stuff, and I think this PR could be reopened, so the comments should be visible to future reviewers. > I hope we can revisit this with you for Apache Spark 4.1.0 timeframe But given this should be done in 4.1, so let's focus on SPARK-51029 and move discussion to https://github.com/apache/spark/pull/49725 for now? -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
