zabetak commented on a change in pull request #1347:
URL: https://github.com/apache/hive/pull/1347#discussion_r468035881
##########
File path:
itests/qtest/src/test/java/org/apache/hadoop/hive/cli/TestTezPerfCliDriver.java
##########
@@ -34,7 +34,7 @@
@RunWith(Parameterized.class)
public class TestTezPerfCliDriver {
- static CliAdapter adapter = new
CliConfigs.TezPerfCliConfig(false).getCliAdapter();
+ static CliAdapter adapter = new
CliConfigs.TezCustomTPCDSCliConfig(false).getCliAdapter();
Review comment:
Well in that case and if we keep the existing perf drivers (which it
might make sense in the short term) it might be better to rename also the test
class (e.g., `TestTezCustomTPCDSCliDriver`) to better reflect what we are
really testing.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
