[Impala-ASF-CR] IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

2017-07-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs .. IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax

[Impala-ASF-CR] IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

2017-07-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs .. Patch Set 14: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5513: Fix display message exception when using invalid KEYVAL

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has posted comments on this change. Change subject: IMPALA-5513: Fix display message exception when using invalid KEYVAL .. Patch Set 4: (2 comments) I have modified the code according to your opinion. I think

[Impala-ASF-CR] IMPALA-5513: Fix display message exception when using invalid KEYVAL

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has uploaded a new patch set (#4). Change subject: IMPALA-5513: Fix display message exception when using invalid KEYVAL .. IMPALA-5513: Fix display message exception when using invalid KEYVAL Function

[Impala-ASF-CR] IMPALA-5513: Fix display message exception when using invalid KEYVAL

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has uploaded a new patch set (#3). Change subject: IMPALA-5513: Fix display message exception when using invalid KEYVAL .. IMPALA-5513: Fix display message exception when using invalid KEYVAL Function

[Impala-ASF-CR] IMPALA-5507: Add clear description to help information of KEYVAL option

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has posted comments on this change. Change subject: IMPALA-5507: Add clear description to help information of KEYVAL option .. Patch Set 4: (3 comments) I have modified the code according to your opinion, please

[Impala-ASF-CR] IMPALA-5507: Add clear description to help information of KEYVAL option

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has uploaded a new patch set (#4). Change subject: IMPALA-5507: Add clear description to help information of KEYVAL option .. IMPALA-5507: Add clear description to help information of KEYVAL option Help information

[Impala-ASF-CR] IMPALA-2636: HS2 GetTables() returns TABLE TYPE as TABLE for VIEW

2017-07-03 Thread sandeep akinapelli (Code Review)
sandeep akinapelli has uploaded a new change for review. http://gerrit.cloudera.org:8080/7353 Change subject: IMPALA-2636: HS2 GetTables() returns TABLE_TYPE as TABLE for VIEW .. IMPALA-2636: HS2 GetTables() returns

[Impala-ASF-CR] IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

2017-07-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs .. Patch Set 14: Build started:

[Impala-ASF-CR] IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

2017-07-03 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs .. Patch Set 14: Code-Review+2 PS14 removes a newline in case of empty coordinator error

[Impala-ASF-CR] IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

2017-07-03 Thread Lars Volker (Code Review)
Hello Impala Public Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7155 to look at the new patch set (#14). Change subject: IMPALA-5427: Fix race between CRS::UpdateQueryStatus() and beeswax RPCs

[Impala-ASF-CR] IMPALA-5513: Fix display message exception when using invalid KEYVAL

2017-07-03 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5513: Fix display message exception when using invalid KEYVAL .. Patch Set 2: (2 comments) Thank you for your patch. You might get faster reviews in the

[Impala-ASF-CR] IMPALA-5507: Add clear description to help information of KEYVAL option

2017-07-03 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5507: Add clear description to help information of KEYVAL option .. Patch Set 3: (4 comments) Thank you for sending this patch. A few comments:

[Impala-ASF-CR] IMPALA-5036: Parquet count star optimization

2017-07-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5036: Parquet count star optimization .. Patch Set 9: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/820/ --

[Impala-ASF-CR] IMPALA-5514: Add the joint judgment of ldap password cmd and ldap

2017-07-03 Thread Donghui Xu (Code Review)
Donghui Xu has posted comments on this change. Change subject: IMPALA-5514: Add the joint judgment of ldap_password_cmd and ldap .. Patch Set 2: Can anyone review this issue please? Thanks. -- To view, visit