Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9461 )
Change subject: IMPALA-2567: Enable KRPC by default ...................................................................... Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/9461/3/bin/start-impala-cluster.py File bin/start-impala-cluster.py: http://gerrit.cloudera.org:8080/#/c/9461/3/bin/start-impala-cluster.py@57 PS3, Line 57: , help > thrift is always enabled. How about "Enable Thrift DataStream service ..." Done http://gerrit.cloudera.org:8080/#/c/9461/3/tests/common/test_skip.py File tests/common/test_skip.py: http://gerrit.cloudera.org:8080/#/c/9461/3/tests/common/test_skip.py@35 PS3, Line 35: assert not pytest.config.option.test_no_krpc : : @SkipIf.not_thrift : def test_skip_if_not_thrift(self): : assert pytest.config.option.test_no_krpc > these look backwards. don't you have to move the 'not'? Oops. Fixed. http://gerrit.cloudera.org:8080/#/c/9461/3/tests/conftest.py File tests/conftest.py: http://gerrit.cloudera.org:8080/#/c/9461/3/tests/conftest.py@120 PS3, Line 120: all tests with > Thrift DataStream service enabled. Done -- To view, visit http://gerrit.cloudera.org:8080/9461 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iae736c1c1351758969b4d84e34fc5b2d048660a0 Gerrit-Change-Number: 9461 Gerrit-PatchSet: 4 Gerrit-Owner: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Comment-Date: Fri, 02 Mar 2018 22:11:19 +0000 Gerrit-HasComments: Yes