Michael Brown has posted comments on this change. Change subject: IMPALA-3739: Enable stress tests on Kudu ......................................................................
Patch Set 5: (2 comments) It seems regardless of database, Kudu tables live in the same Kudu namespace. In order to have TPCH and TPCDS data both loaded, at least one of the customer tables must have a different name. But also, it's often convenient to have the same data of varying scales loaded on a stress cluster for various experiments. Since we solve that problem by including the data scale in the database name, I think we could solve these problems in Kudu by including the Impala database as a prefix to each Kudu table. Suggestion: prepend all the Kudu table names with the database name. http://gerrit.cloudera.org:8080/#/c/4327/5/testdata/datasets/tpcds/tpcds_kudu_template.sql File testdata/datasets/tpcds/tpcds_kudu_template.sql: PS5, Line 469: 'kudu.table_name'='customer', Here's one kudu table called 'customer'. Can this (and others) be expanded to "{target_db_name}_customer" ? http://gerrit.cloudera.org:8080/#/c/4327/5/testdata/datasets/tpch/tpch_kudu_template.sql File testdata/datasets/tpch/tpch_kudu_template.sql: PS5, Line 185: 'kudu.table_name' = 'customer', Here's another kudu table called "customer". -- To view, visit http://gerrit.cloudera.org:8080/4327 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3c9fc3dae24b761f031ee8e014bd611a49029d34 Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Dimitris Tsirogiannis <dtsirogian...@cloudera.com> Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogian...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Michael Brown <mi...@cloudera.com> Gerrit-HasComments: Yes