[Impala-ASF-CR] IMPALA-4862: make resource profile consistent with backend behaviour

2017-07-10 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4862: make resource profile consistent with backend behaviour .. Patch Set 14: (1 comment)

[Impala-ASF-CR] IMPALA-5333: [DOCS] Document Impala ADLS support

2017-07-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5333: [DOCS] Document Impala ADLS support .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7175 To unsubscribe,

[Impala-ASF-CR] IMPALA-5394: Handle blocked HS2 connections

2017-07-10 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5394: Handle blocked HS2 connections .. Patch Set 3: Hi John - Thanks for taking the time to make this change! I'll probably do the first review round

[Impala-ASF-CR] IMPALA-5394: Handle blocked HS2 connections

2017-07-10 Thread John Sherman (Code Review)
John Sherman has posted comments on this change. Change subject: IMPALA-5394: Handle blocked HS2 connections .. Patch Set 3: Giving this a bump, since my updated patch might have slipped under the radar during the holiday.

[Impala-ASF-CR] IMPALA-5333: [DOCS] Document Impala ADLS support

2017-07-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5333: [DOCS] Document Impala ADLS support .. Patch Set 4: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/137/ -- To view,

[Impala-ASF-CR] IMPALA-5333: [DOCS] Document Impala ADLS support

2017-07-10 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-5333: [DOCS] Document Impala ADLS support .. Patch Set 4: Code-Review+2 Bumping to +2 based on Sailesh's and Laurel's comments. -- To view, visit

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 3: Yeah I think we should avoid reporting two different values for sure. Maybe we should

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 3: > Yeah I think we should avoid reporting two different values for > sure. Maybe we

[Impala-ASF-CR] IMPALA-5333: [DOCS] Document Impala ADLS support

2017-07-10 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: IMPALA-5333: [DOCS] Document Impala ADLS support .. Patch Set 4: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7175/3/docs/topics/impala_adls.xml

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 3: > Thanks Matt, that's a good point. We should probably just display > that metric's

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 3: Thanks Matt, that's a good point. We should probably just display that metric's value

[Impala-ASF-CR] IMPALA-5333: [DOCS] Document Impala ADLS support

2017-07-10 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5333: [DOCS] Document Impala ADLS support .. Patch Set 4: Code-Review+1 Thanks for working on this John! LGTM Feel free to upgrade to a +2 if Laurel has

[native-toolchain-CR] Bump Kudu version to 1070e76

2017-07-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 1070e76 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7384 To unsubscribe, visit

[native-toolchain-CR] Bump Kudu version to 1070e76

2017-07-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has submitted this change and it was merged. Change subject: Bump Kudu version to 1070e76 .. Bump Kudu version to 1070e76 Change-Id: I0f3c99544a35ee47502e1b3aba8a9dfb048ac0ca --- M buildall.sh 1 file changed, 1

[native-toolchain-CR] Bump Kudu version to 1070e76

2017-07-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump Kudu version to 1070e76 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7384 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 3: We also have a metric (see http endpoint /metrics) called "impala-server.start-time",

[Impala-ASF-CR] IMPALA-5623: Fix lag() on STRING cols to release UDF mem

2017-07-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5623: Fix lag() on STRING cols to release UDF mem .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/834/ --

[Impala-ASF-CR] single node perf run.py: clean up newly-added testdata

2017-07-10 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: single_node_perf_run.py: clean up newly-added testdata .. Patch Set 2: Code-Review+2 rebase carry +2 -- To view, visit http://gerrit.cloudera.org:8080/7291 To

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

2017-07-10 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 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7179 To

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

2017-07-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5507: Add clear description to help information of KEYVAL option .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#3). Change subject: IMPALA-5511: Add process start time to debug web page .. IMPALA-5511: Add process start time to debug web page Read the start date and time of the impalad,

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#3). Change subject: IMPALA-5511: Add process start time to debug web page .. IMPALA-5511: Add process start time to debug web page Read the start date and time of the impalad,

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#2). Change subject: IMPALA-5511: Add process start time to debug web page .. IMPALA-5511: Add process start time to debug web page Read the start date and time of the impalad,

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7363/1/be/src/util/process-state-info.cc

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

2017-07-10 Thread Donghui Xu (Code Review)
Donghui Xu has uploaded a new patch set (#10). 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-5612: join inversion should factor in parallelism

2017-07-10 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5612: join inversion should factor in parallelism .. Patch Set 3: Code-Review+1 (4 comments)

[Impala-ASF-CR] single node perf run.py: clean up newly-added testdata

2017-07-10 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: single_node_perf_run.py: clean up newly-added testdata .. Patch Set 1: Verified-1 Build failed:

<    1   2