[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#7). Change subject: IMPALA-5500: Reduce catalog update topic size .. IMPALA-5500: Reduce catalog update topic size Problem: IMPALA-4029 introduced the use of the flatbuffers s

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#6). Change subject: IMPALA-5500: Reduce catalog update topic size .. IMPALA-5500: Reduce catalog update topic size Problem: IMPALA-4029 introduced the use of the flatbuffers s

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc F

[Impala-ASF-CR] IMPALA-5317: add DATE TRUNC() function

2017-06-27 Thread sandeep akinapelli (Code Review)
sandeep akinapelli has uploaded a new change for review. http://gerrit.cloudera.org:8080/7313 Change subject: IMPALA-5317: add DATE_TRUNC() function .. IMPALA-5317: add DATE_TRUNC() function Added a UDF builtin function date_

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5567: race in fragment instance teardown .. IMPALA-5567: race in fragment instance teardown The bug is that PlanRootSink::GetNext() calls

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7275 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 This change fixes the frontend tests to make

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7073 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc File be/src/ca

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Zach Amsden (Code Review)
Zach Amsden has abandoned this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/6477 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Ger

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Code-Review-1 I think we have more important things to do and there are probably even more egregious tests. N

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

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4862: make resource profile consistent with backend behaviour .. Patch Set 7: Rebased onto the buffer resizing patchset and updated planner test output.

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

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#7). Change subject: IMPALA-4862: make resource profile consistent with backend behaviour .. IMPALA-4862: make resource profile consistent with backend behaviour This moves away from

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc File be/sr

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc F

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Do we plan to move forward with this? -- To view, visit http://gerrit.cloudera.org:8080/6477 To unsubscribe, vi

[Impala-ASF-CR] IMPALA-3504: UDF for current timestamp in UTC

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3504: UDF for current timestamp in UTC .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7203 To unsubscribe, visit http://gerr

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc File be/sr

[Impala-ASF-CR] IMPALA-5539: Kudu timestamp scans wrong with -use local tz for unix ts

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5539: Kudu timestamp scans wrong with -use_local_tz_for_unix_ts .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7311/1//COMMIT_MSG Commit M

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: Reduce the frequency of fault injection .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7310/1/be/src/runtime/backend-client.h File be

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: Reduce the frequency of fault injection .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7310/1/be/src/runtime/backend-client.h File be

[Impala-ASF-CR] IMPALA-5489: Improve Sentry authorization for Kudu tables

2017-06-27 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-5489: Improve Sentry authorization for Kudu tables .. Patch Set 1: (2 comments) Just a couple of nits. http://gerrit.cloudera.org:8080/#/c/7307/1/fe/src

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc Fi

[Impala-ASF-CR] IMPALA-5539: Kudu timestamp scans wrong with -use local tz for unix ts

2017-06-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7311 Change subject: IMPALA-5539: Kudu timestamp scans wrong with -use_local_tz_for_unix_ts .. IMPALA-5539: Kudu timestamp scans wrong with

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc File be/s

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: Bump toolchain version .. Bump toolchain version Bump the versions to keep things in sync with native-toolchain master. The new toolchain versi

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7306 To unsubscribe, visit http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/797/ -- To view,

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 6: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/7275 To unsubscribe, v

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, Lars Volker, Matthew Jacobs, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7275 to look at the new patch set (#5). Change subject: IMPALA-5567: race in fragment instance teardown

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7275/4/be/src/runtime/runtime-state.cc File be/s

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: Reduce the frequency of fault injection .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7310/1/be/src/runtime/backend-client.h File be

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: Reduce the frequency of fault injection .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7310/1/be/src/runtime/backend-client.h File be

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5588: Reduce the frequency of fault injection .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7310/1/be/src/runtime/backend-client.h File

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7268/5/be/src/catalog/catalog-server.cc File be/src/c

[Impala-ASF-CR] IMPALA-5588: RPC end to end tests: protect against eval of None

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: RPC end to end tests: protect against eval of None .. Patch Set 1: Thanks. Fix incorporated in this fix: https://gerrit.cloudera.org/#/c/7310/ -- To

[Impala-ASF-CR] IMPALA-5588: Reduce the frequency of fault injection

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new change for review. http://gerrit.cloudera.org:8080/7310 Change subject: IMPALA-5588: Reduce the frequency of fault injection .. IMPALA-5588: Reduce the frequency of fault injection Previously, th

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#5). Change subject: IMPALA-5500: Reduce catalog update topic size .. IMPALA-5500: Reduce catalog update topic size Problem: IMPALA-4029 introduced the use of the flatbuffers s

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/4/be/src/catalog/catalog-server.cc Fi

[Impala-ASF-CR] IMPALA-5588: RPC end to end tests: protect against eval of None

2017-06-27 Thread Michael Brown (Code Review)
Michael Brown has abandoned this change. Change subject: IMPALA-5588: RPC end to end tests: protect against eval of None .. Abandoned Michael is actively working on this, which I wasn't sure about. He'll fix this in the real I

[Impala-ASF-CR] IMPALA-5588: RPC end to end tests: protect against eval of None

2017-06-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5588: RPC end to end tests: protect against eval of None .. Patch Set 1: Can I fold this into the other change for IMPALA-5588 ? -- To view, visit http://g

[Impala-ASF-CR] IMPALA-5588: RPC end to end tests: protect against eval of None

2017-06-27 Thread Michael Brown (Code Review)
Michael Brown has uploaded a new change for review. http://gerrit.cloudera.org:8080/7308 Change subject: IMPALA-5588: RPC end to end tests: protect against eval of None .. IMPALA-5588: RPC end to end tests: protect against eva

[Impala-ASF-CR] IMPALA-5489: Improve Sentry authorization for Kudu tables

2017-06-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/7307 Change subject: IMPALA-5489: Improve Sentry authorization for Kudu tables .. IMPALA-5489: Improve Sentry authorization for Kudu tables

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 10: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/796/ -- To view,

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 10: Code-Review+2 Rebased, carrying Alex's +2. -- To view, visit http://gerrit.cloudera.org:8080/707

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7268/4/be/src/catalog/catalog-server.cc File be/src/ca

[Impala-ASF-CR] IMPALA-5160: adjust spill buffer size based on planner estimates

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5160: adjust spill buffer size based on planner estimates .. IMPALA-5160: adjust spill buffer size based on planner estimates Scale down

[Impala-ASF-CR] IMPALA-5160: adjust spill buffer size based on planner estimates

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5160: adjust spill buffer size based on planner estimates .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6963 To u

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 9: Thank you for the review. I will rebase the change and make sure one more time that tests pass on

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7073 To unsubscribe, visit http://ge

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#9). Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 This change fixes the frontend tests to make them run on Java 8, b

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#4). Change subject: IMPALA-5500: Reduce catalog update topic size .. IMPALA-5500: Reduce catalog update topic size Problem: IMPALA-4029 introduced the use of the flatbuffers s

[Impala-ASF-CR] IMPALA-5500: Reduce catalog update topic size

2017-06-27 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-5500: Reduce catalog update topic size .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/7268/3/be/src/catalog/catalog-server.cc F

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/795/ -- To view, visit http://gerrit.cloud

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7275/4/be/src/runtime/runtime-state.cc File be/src/r

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/7275/4/be/src/runtime/runtime-state.cc

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis/C

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 2: Code-Review+1 Thanks! -- To view, visit http://gerrit.cloudera.org:8080/7306 To unsubscribe, visit http://gerrit.cloudera.

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analysis

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analys

[Impala-ASF-CR] IMPALA-4687: Get Impala working against HBase 2.0

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4687: Get Impala working against HBase 2.0 .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7277 To unsubscribe, visit http://

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/7073/7/fe/src/main/java/org/apache/impala/analys

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. Patch Set 7: (3 comments) Thank you for the review. Please see my comments and PS8. http://gerrit.cloudera.org

[Impala-ASF-CR] IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#8). Change subject: IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 .. IMPALA-3643/IMPALA-5344: Fix FE tests on Java 8 This change fixes the frontend tests to make them run on Java 8, b

[Impala-ASF-CR] IMPALA-5240: Allow config of number of disk I/O threads per disk type

2017-06-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5240: Allow config of number of disk I/O threads per disk type .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7232/5/be/src/runtime/d

[Impala-ASF-CR] IMPALA-4687: Get Impala working against HBase 2.0

2017-06-27 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4687: Get Impala working against HBase 2.0 .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7277 To unsubscribe, vi

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7306 To unsubscribe, visit http://gerrit.cloudera.org:80

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: Bump toolchain version .. Bump toolchain version Bump the versions to keep things in sync with native-toolchain master. The new toolchain version includes fixes t

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7306/1//COMMIT_MSG Commit Message: Line 9: Bump the versions to keep thi

[Impala-ASF-CR] IMPALA-5560: always store CHAR(N) inline in tuple

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-5560: always store CHAR(N) inline in tuple .. IMPALA-5560: always store CHAR(N) inline in tuple This is done to simplify the CHAR(N) logic. I believe this i

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Bump toolchain version .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7306/1//COMMIT_MSG Commit Message: Line 9: Bump the versions to keep thing

[Impala-ASF-CR] IMPALA-4687: Get Impala working against HBase 2.0

2017-06-27 Thread Joe McDonnell (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7277 to look at the new patch set (#3). Change subject: IMPALA-4687: Get Impala working against HBase 2.0 .. IMP

[Impala-ASF-CR] Bump toolchain version

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/7306 Change subject: Bump toolchain version .. Bump toolchain version Bump the versions to keep things in sync with native-toolchain master.

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7275 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-4687: Get Impala working against HBase 2.0

2017-06-27 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-4687: Get Impala working against HBase 2.0 .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7277/1/be/src/exec/hbase-table-scanner.cc Fil

[native-toolchain-CR] More GCC 7 fixes

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: More GCC 7 fixes .. Patch Set 4: Verified+1 Build ID is 415-a56999c8bb -- To view, visit http://gerrit.cloudera.org:8080/7220 To unsubscribe, visit http://gerr

[native-toolchain-CR] More GCC 7 fixes

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. Change subject: More GCC 7 fixes .. More GCC 7 fixes My previous patch didn't fix all the build issues - I was building in a pre-existing workspace with some packages a

[Impala-ASF-CR] IMPALA-5280: Coalesce chains of OR conditions to an IN predicate

2017-06-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5280: Coalesce chains of OR conditions to an IN predicate .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/7110/8/testdata/workloads/functiona

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5567: race in fragment instance teardown .. Patch Set 2: (1 comment) Ok, I think we're back to the more minimal fix. Does this seem ok to everyone? http

[Impala-ASF-CR] IMPALA-5567: race in fragment instance teardown

2017-06-27 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7275 to look at the new patch set (#3). Change subject: IMPALA-5567: race in fragment instance teardown ...

[Impala-ASF-CR] IMPALA-5280: Coalesce chains of OR conditions to an IN predicate

2017-06-27 Thread sandeep akinapelli (Code Review)
sandeep akinapelli has posted comments on this change. Change subject: IMPALA-5280: Coalesce chains of OR conditions to an IN predicate .. Patch Set 8: Fixed test cases by new checking in the new plan after OR to IN conversion

[Impala-ASF-CR] IMPALA-5280: Coalesce chains of OR conditions to an IN predicate

2017-06-27 Thread sandeep akinapelli (Code Review)
Hello Impala Public Jenkins, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7110 to look at the new patch set (#8). Change subject: IMPALA-5280: Coalesce chains of OR conditions to an IN predicate

[Impala-ASF-CR] IMPALA-5160: adjust spill buffer size based on planner estimates

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5160: adjust spill buffer size based on planner estimates .. Patch Set 12: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/7

[Impala-ASF-CR] IMPALA-5571: Fix authorization tests

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5571: Fix authorization tests .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7299 To unsubscribe, visit http://gerr

[Impala-ASF-CR] IMPALA-5571: Fix authorization tests

2017-06-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5571: Fix authorization tests .. IMPALA-5571: Fix authorization tests Issue: With the fix for IMPALA-5549 the catalog cache of an impalad