bhasudha commented on a change in pull request #1003: [HUDI-218] Adding Presto
support to Integration Test
URL: https://github.com/apache/incubator-hudi/pull/1003#discussion_r344420778
##########
File path: hudi-integ-test/src/test/java/org/apache/hudi/integ/ITTestBase.java
##########
@@ -63,6 +64,9 @@
protected static final String HUDI_UTILITIES_BUNDLE =
HOODIE_WS_ROOT +
"/docker/hoodie/hadoop/hive_base/target/hoodie-utilities.jar";
protected static final String HIVE_SERVER_JDBC_URL =
"jdbc:hive2://hiveserver:10000";
+ protected static final String PRESTO_COORDINATOR_URL =
"presto-coordinator-1:8090";
+ protected static final String PRESTO_USE_HIVE_DEFAULT = "use hive.default;";
Review comment:
remove this line ? Seems like you are already passing catalog and schema to
the presto command line
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
