Gowthami Bisati has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24649
Change subject: IMPALA-5406: Move Kudu SHOW CREATE TABLE tests to .test file ...................................................................... IMPALA-5406: Move Kudu SHOW CREATE TABLE tests to .test file Moving SHOW CREATE TABLE tests from inline Python string definitions in test_kudu.py into dedicated .test SQL files makes test cases much easier to maintain, review, and keep in sync with Impala SQL behavior. Change: - Removed class TestShowCreateTable from test_kudu.py. - Added test runner for kudu-show-create-table.test to test_kudu.py. - Added new test file under: testdata/workloads/functional-query/queries/QueryTest/kudu-show-create-table.test Testing Performed: - Verified frontend changes compile cleanly using checkstyle. - Ran mvn clean package -DskipTests (BUILD SUCCESS). - Ran focus testing suite: mvn test -Dtest=ToSqlTest (BUILD SUCCESS). - Ran idempotency integrity validation suite: mvn test -Dtest=ToSqlUtilsTest (BUILD SUCCESS). - Ran flake8 tests/query_test/test_kudu.py. - Ran impala-py.test tests/query_test/test_kudu.py. - Ran impala-py.test tests/metadata/test_ddl.py. - Ran bin/impala-py.test --exploration_strategy=exhaustive tests/custom_cluster/test_workload_mgmt_init.py. - Ran python3 bin/jenkins/critique-gerrit-review.py --dryrun. Change-Id: Ie29e770a5546018c7dcd94b7018bd3c619639da2 Signed-off-by: Gowthami Bisati <[email protected]> --- A testdata/workloads/functional-query/queries/QueryTest/kudu-show-create-table.test M tests/query_test/test_kudu.py 2 files changed, 173 insertions(+), 316 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/24649/1 -- To view, visit http://gerrit.cloudera.org:8080/24649 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ie29e770a5546018c7dcd94b7018bd3c619639da2 Gerrit-Change-Number: 24649 Gerrit-PatchSet: 1 Gerrit-Owner: Gowthami Bisati <[email protected]>
