Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4365: Enabling end-to-end tests on a remote cluster ......................................................................
Patch Set 13: (3 comments) http://gerrit.cloudera.org:8080/#/c/4769/13//COMMIT_MSG Commit Message: Line 51: still problems to work out with the remote data load script itself. Did you try loading data on a remote cluster and running tests on in with the scripts in this patch? http://gerrit.cloudera.org:8080/#/c/4769/13/bin/remote_data_load.py File bin/remote_data_load.py: Line 534: sys.exit(1) In general, I think it's a bad practice to call sys.exit inside functions. It's better to raise an exception and if it does not get caught on the outside, then exit will be triggered. http://gerrit.cloudera.org:8080/#/c/4769/13/testdata/datasets/functional/schema_constraints.csv File testdata/datasets/functional/schema_constraints.csv: PS13, Line 120: Wide tables fail due to the SERDEPROPERTIES limits is this a new issue? Is it specific to remote data loading? I looked at HIVE-1364 it says it got fixed a few years ago. -- To view, visit http://gerrit.cloudera.org:8080/4769 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1f443a1728a1d28168090c6f54e82dec2cb073e9 Gerrit-PatchSet: 13 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: David Knupp <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Harrison Sheinblatt <[email protected]> Gerrit-Reviewer: Martin Grund <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-HasComments: Yes
