[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()

[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-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

[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.

[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

[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,

[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

[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

[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

[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

[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

[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.

[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

[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

[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

[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

[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,

[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

[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

[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

[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

[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

[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,

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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)

[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,

[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

[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

[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

[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

[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)

[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)

[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

[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)

[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)

[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

[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)

[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.

[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,

[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)

[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,

[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

[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

[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

[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

[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

[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 ..

[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

[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

[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

[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

[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)

[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?

[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

[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:

[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

[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

[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 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/7268/3/be/src/catalog/catalog-server.cc File