Impala Public Jenkins has submitted this change and it was merged. Change subject: Revert "IMPALA-5589: change "set" in impala-shell to show empty string for unset query options" ......................................................................
Revert "IMPALA-5589: change "set" in impala-shell to show empty string for unset query options" Due to re-use of connections in the test infrastructure, this commit is causing ASAN failures in the builds. That is being worked out as part of IMPALA-5908, but, in the meanwhile, it's prudent to revert. This reverts commit 387bde0639ffd8ef580ccbf727152954e62bacbe. Change-Id: I41bc8ab0f1df45bbd311030981a7c18989c2edc8 Reviewed-on: http://gerrit.cloudera.org:8080/8087 Reviewed-by: Dan Hecht <dhe...@cloudera.com> Tested-by: Impala Public Jenkins --- M be/src/service/query-options-test.cc M be/src/service/query-options.cc M be/src/service/query-options.h M testdata/workloads/functional-query/queries/QueryTest/set.test M tests/hs2/hs2_test_suite.py M tests/hs2/test_hs2.py M tests/shell/test_shell_commandline.py 7 files changed, 60 insertions(+), 106 deletions(-) Approvals: Impala Public Jenkins: Verified Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8087 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41bc8ab0f1df45bbd311030981a7c18989c2edc8 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins