Re: [PR] [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name [spark]

2024-01-10 Thread via GitHub
dongjoon-hyun commented on PR #44657: URL: https://github.com/apache/spark/pull/44657#issuecomment-1884579868 Thank you all! Merged to master. -- 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

Re: [PR] [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name [spark]

2024-01-10 Thread via GitHub
dongjoon-hyun closed pull request #44657: [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name URL: https://github.com/apache/spark/pull/44657 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name [spark]

2024-01-09 Thread via GitHub
yaooqinn commented on PR #44657: URL: https://github.com/apache/spark/pull/44657#issuecomment-1884206112 LGTM. Taking into account the gold files, I'm fine with leaving it as is for now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name [spark]

2024-01-09 Thread via GitHub
dongjoon-hyun commented on PR #44657: URL: https://github.com/apache/spark/pull/44657#issuecomment-1884183494 cc @yaooqinn -- 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.

[PR] [SPARK-46652][SQL][TESTS] Remove `Snappy` from `TPCDSQueryBenchmark` benchmark case name [spark]

2024-01-09 Thread via GitHub
dongjoon-hyun opened a new pull request, #44657: URL: https://github.com/apache/spark/pull/44657 … ### What changes were proposed in this pull request? ### Why are the changes needed? ### Does this PR introduce _any_ user-facing change?