[Impala-ASF-CR] IMPALA-6173: Fix SHOW CREATE TABLE for unpartitioned Kudu tables

2017-11-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8506 ) Change subject: IMPALA-6173: Fix SHOW CREATE TABLE for unpartitioned Kudu tables .. Patch Set 1: GVO failure was unrelated.

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 14: (7 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-09 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#14). Change subject: IMPALA-4252:

[Impala-ASF-CR] IMPALA-6173: Fix SHOW CREATE TABLE for unpartitioned Kudu tables

2017-11-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8506 Change subject: IMPALA-6173: Fix SHOW CREATE TABLE for unpartitioned Kudu tables .. IMPALA-6173: Fix SHOW CREATE TABLE for

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-08 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#13). Change subject: IMPALA-4252:

[Impala-ASF-CR] IMPALA-4591: Bound Kudu client error mem usage

2017-11-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8464 ) Change subject: IMPALA-4591: Bound Kudu client error mem usage .. Patch Set 2: (8 comments)

[Impala-ASF-CR] IMPALA-4591: Bound Kudu client error mem usage

2017-11-08 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8464 to look at the new patch set (#2). Change subject: IMPALA-4591: Bound Kudu client error mem usage .. IMPALA-4591:

[Impala-ASF-CR] IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources"

2017-11-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8499 ) Change subject: IMPALA-6171: Revert "IMPALA-1575: part 2: yield admission control resources" .. Patch Set 1: Code-Review+2

[Impala-ASF-CR] Bump Kudu version to 1520b39

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8492 Change subject: Bump Kudu version to 1520b39 .. Bump Kudu version to 1520b39 Change-Id:

[native-toolchain-CR] Bump Kudu version to 1520b39

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8491 ) Change subject: Bump Kudu version to 1520b39 .. Bump Kudu version to 1520b39 Change-Id:

[native-toolchain-CR] Bump Kudu version to 1520b39

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8491 ) Change subject: Bump Kudu version to 1520b39 .. Patch Set 1: Verified+1 -- To view, visit

[native-toolchain-CR] Bump Kudu version to 1520b39

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8491 ) Change subject: Bump Kudu version to 1520b39 .. Patch Set 1:

[native-toolchain-CR] Bump Kudu version to 1520b39

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8491 Change subject: Bump Kudu version to 1520b39 .. Bump Kudu version to 1520b39 Change-Id:

[Impala-ASF-CR] IMPALA-2248: Make idle session timeout a query option

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8490 ) Change subject: IMPALA-2248: Make idle_session_timeout a query option .. Patch Set 3: (4 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 11: (8 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-07 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#12). Change subject: IMPALA-4252:

[Impala-ASF-CR] IMPALA-4591: Bound Kudu client error mem usage

2017-11-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8464 Change subject: IMPALA-4591: Bound Kudu client error mem usage .. IMPALA-4591: Bound Kudu client error mem usage Previously,

[Impala-ASF-CR] IMPALA-6151: add query-level fragment/backend counters

2017-11-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8461 ) Change subject: IMPALA-6151: add query-level fragment/backend counters .. Patch Set 1: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 11: (19 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-03 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#11). Change subject: IMPALA-4252: Min-max

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-03 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#10). Change subject: IMPALA-4252: Min-max

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#9). Change subject: IMPALA-4252: Min-max runtime filters

[Impala-ASF-CR] IMPALA-6127: Fix timeout in TestRuntimeFilter.test wait time

2017-10-31 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8427 Change subject: IMPALA-6127: Fix timeout in TestRuntimeFilter.test_wait_time .. IMPALA-6127: Fix timeout in

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-27 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 8: TPCDS results:

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 8: > > Patch Set 7: > > > > > > Patch Set 7: >

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 8: (10 comments) > I noticed that -

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-26 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#8). Change subject: IMPALA-4252: Min-max runtime filters

[Impala-ASF-CR] IMPALA-6099: Fix crash in CheckForAlwaysFalse()

2017-10-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8369 ) Change subject: IMPALA-6099: Fix crash in CheckForAlwaysFalse() .. Patch Set 1: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8358 ) Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-23 Thread Thomas Tauber-Marshall (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8358 to look at the new patch set (#2). Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. IMPALA-6004:

[Impala-ASF-CR] IMPALA-6004: Fix test row filters failure on ASAN

2017-10-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8358 Change subject: IMPALA-6004: Fix test_row_filters failure on ASAN .. IMPALA-6004: Fix test_row_filters failure on ASAN "Test

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 7: Perf results:

[Impala-ASF-CR] IMPALA-5307: Part 2: copy out strings in uncompressed Avro

2017-10-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8146 ) Change subject: IMPALA-5307: Part 2: copy out strings in uncompressed Avro .. Patch Set 10: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5307: Part 2: copy out strings in uncompressed Avro

2017-10-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8146 ) Change subject: IMPALA-5307: Part 2: copy out strings in uncompressed Avro .. Patch Set 9: (4 comments) Looks good

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-18 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#7). Change subject: IMPALA-4252: Min-max runtime filters for Kudu

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 6: (18 comments)

[Impala-ASF-CR] IMPALA-5789: Add always false flag in bloom filter

2017-10-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8170 ) Change subject: IMPALA-5789: Add always_false flag in bloom filter .. Patch Set 4: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6027: Retry downloading toolchain components.

2017-10-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8258 ) Change subject: IMPALA-6027: Retry downloading toolchain components. .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5789: Add always false flag in bloom filter

2017-10-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8170 ) Change subject: IMPALA-5789: Add always_false flag in bloom filter .. Patch Set 3: (6 comments) This is definitely going to

[Impala-ASF-CR] IMPALA-5529: [DOCS] New trunc() signatures

2017-10-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8189 ) Change subject: IMPALA-5529: [DOCS] New trunc() signatures .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-2758: Change BufferedTupleStream::GetRows to returning multi batches

2017-10-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Change BufferedTupleStream::GetRows to returning multi batches .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 6: (32 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-10-06 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Tim Armstrong, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#6). Change subject: IMPALA-4252: Min-max runtime filters for Kudu

[Impala-ASF-CR] IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax

2017-10-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8191 ) Change subject: IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax .. Patch Set 4: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax

2017-10-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8191 ) Change subject: IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax .. Patch Set 4: (1 comment)

[Impala-ASF-CR] Bump Kudu version to bec2a24

2017-10-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8207 ) Change subject: Bump Kudu version to bec2a24 .. Patch Set 2: Code-Review+2 carrying forward -- To view, visit

[Impala-ASF-CR] IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax

2017-10-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8191 ) Change subject: IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax .. Patch Set 3: (1 comment)

[Impala-ASF-CR] Bump Kudu version to bec2a24

2017-10-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8207 Change subject: Bump Kudu version to bec2a24 .. Bump Kudu version to bec2a24 Change-Id:

[Impala-ASF-CR] IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax

2017-10-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8191 ) Change subject: IMPALA-4622: [DOCS] New Kudu ALTER TABLE syntax .. Patch Set 2: (4 comments)

[native-toolchain-CR] Bump Kudu version to bec2a24

2017-10-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8192 ) Change subject: Bump Kudu version to bec2a24 .. Bump Kudu version to bec2a24 Change-Id:

[native-toolchain-CR] Bump Kudu version to bec2a24

2017-10-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8192 ) Change subject: Bump Kudu version to bec2a24 .. Patch Set 1: Verified+1 -- To view, visit

[native-toolchain-CR] Bump Kudu version to bec2a24

2017-10-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8192 ) Change subject: Bump Kudu version to bec2a24 .. Patch Set 1:

[native-toolchain-CR] Bump Kudu version to bec2a24

2017-10-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8192 Change subject: Bump Kudu version to bec2a24 .. Bump Kudu version to bec2a24 Change-Id:

[Impala-ASF-CR] IMPALA-5383: [DOCS] Document unpartitioned Kudu tables

2017-10-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8180 ) Change subject: IMPALA-5383: [DOCS] Document unpartitioned Kudu tables .. Patch Set 3: (1 comment) > Thomas, can I tag you

[Impala-ASF-CR] IMPALA-4252: Move runtime filters to ScanNode

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8148 ) Change subject: IMPALA-4252: Move runtime filters to ScanNode .. Patch Set 2: (3 comments)

[Impala-ASF-CR] IMPALA-4252: Move runtime filters to ScanNode

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8148 to look at the new patch set (#2). Change subject: IMPALA-4252: Move runtime filters to ScanNode .. IMPALA-4252:

[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 4: Code-Review+2 Carrying forward +2. Gvo failed due

[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Hello Lars Volker, Alex Behm, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8169 to look at the new patch set (#4). Change subject: IMPALA-5994: Lower case struct-field names

[Impala-ASF-CR] IMPALA-5951: Remove flaky test catalogd timeout

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8154 ) Change subject: IMPALA-5951: Remove flaky test_catalogd_timeout .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5951: Remove flaky test catalogd timeout

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8154 to look at the new patch set (#2). Change subject: IMPALA-5951: Remove flaky test_catalogd_timeout ..

[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names

2017-09-29 Thread Thomas Tauber-Marshall (Code Review)
Hello Lars Volker, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8169 to look at the new patch set (#3). Change subject: IMPALA-5994: Lower case struct-field names ..

[Impala-ASF-CR] IMPALA-5994: Failure in star expansion on struct fields

2017-09-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Failure in star expansion on struct fields .. Patch Set 2: (3 comments)

[Impala-ASF-CR] IMPALA-5994: Failure in star expansion on struct fields

2017-09-28 Thread Thomas Tauber-Marshall (Code Review)
Hello Lars Volker, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8169 to look at the new patch set (#2). Change subject: IMPALA-5994: Failure in star expansion on struct fields

[Impala-ASF-CR] IMPALA-5994: Failure in star expansion on struct fields

2017-09-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8169 Change subject: IMPALA-5994: Failure in star expansion on struct fields .. IMPALA-5994: Failure in star expansion on struct

[Impala-ASF-CR] IMPALA-5951: test catalogd timeout fails to cause expected exception

2017-09-27 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8154 Change subject: IMPALA-5951: test_catalogd_timeout fails to cause expected exception .. IMPALA-5951: test_catalogd_timeout

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 5: (1 comment) > The Parquet writer does

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#5). Change subject: IMPALA-4252: Min-max runtime filters for Kudu

[Impala-ASF-CR] DRAFT - IMPALA-4252: Min-max runtime filters for Kudu

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Lars Volker, Matthew Jacobs, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#4). Change subject: DRAFT - IMPALA-4252: Min-max runtime filters for Kudu

[Impala-ASF-CR] IMPALA-4252: Move runtime filters to ScanNode

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8148 Change subject: IMPALA-4252: Move runtime filters to ScanNode .. IMPALA-4252: Move runtime filters to ScanNode As a

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 ) Change subject: IMPALA-5870: Improve runtime profile for partial sort .. Patch Set 3: The GVO failed because there was a kudu

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-26 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8123 to look at the new patch set (#3). Change subject: IMPALA-5870: Improve runtime profile for partial sort

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 7: Code-Review+2 GVO failed because of

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-25 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Jim Apple, Philip Zeyliger, Tim Armstrong, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#7). Change subject: IMPALA-3360: Codegen inserting into

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8123 ) Change subject: IMPALA-5870: Improve runtime profile for partial sort .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-5870: Improve runtime profile for partial sort

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8123 to look at the new patch set (#2). Change subject: IMPALA-5870: Improve runtime profile for partial sort ..

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8029 ) Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-22 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Jim Apple, Philip Zeyliger, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#6). Change subject: IMPALA-3360: Codegen inserting into runtime filters

[Impala-ASF-CR] IMPALA-5870: Improve explain/profile output for partial sort

2017-09-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8123 Change subject: IMPALA-5870: Improve explain/profile output for partial sort .. IMPALA-5870: Improve explain/profile output for

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-5949: fix test exchange small delay failure

2017-09-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5949: fix test_exchange_small_delay failure .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8111 To

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/8029/3/be/src/exec/CMakeLists.txt

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-20 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#5). Change subject: IMPALA-3360: Codegen inserting into runtime filters ..

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-20 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#4). Change subject: IMPALA-3360: Codegen inserting into runtime filters ..

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-18 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8029 to look at the new patch set (#3). Change subject: IMPALA-3360: Codegen inserting into runtime filters ..

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-3360: Codegen inserting into runtime filters .. IMPALA-3360: Codegen inserting into runtime filters This patch codegens

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3360: Codegen inserting into runtime filters .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-5416: Fix an impala-shell command recursion bug

2017-09-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5416: Fix an impala-shell command recursion bug .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8063 To

[Impala-ASF-CR] IMPALA-5416: Fix an impala-shell command recursion bug

2017-09-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5416: Fix an impala-shell command recursion bug .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8063/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5416: Fix an impala-shell command recursion bug

2017-09-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5416: Fix an impala-shell command recursion bug .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/8063/1//COMMIT_MSG Commit

[Impala-ASF-CR] Bump Kudu version to 3f49724

2017-09-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/8040 Change subject: Bump Kudu version to 3f49724 .. Bump Kudu version to 3f49724 Change-Id:

[Impala-ASF-CR] IMPALA-1767 Adds predicate to test boolean values true, false, unknown.

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-1767 Adds predicate to test boolean values true, false, unknown. .. Patch Set 3: (10 comments)

[native-toolchain-CR] Bump Kudu version to 3f49724

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 3f49724 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8028 To unsubscribe, visit

[native-toolchain-CR] Bump Kudu version to 3f49724

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bump Kudu version to 3f49724 .. Bump Kudu version to 3f49724 Change-Id: I38334bdbaa1e7a6c6f5d595c26efb76efa062305 --- M buildall.sh 1 file

[Impala-ASF-CR] IMPALA-5912: fix crash in trunc(..., "WW") in release build

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5912: fix crash in trunc(..., "WW") in release build .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8015 To

[Impala-ASF-CR] IMPALA-3360: Codegen inserting into runtime filters

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/8029 Change subject: IMPALA-3360: Codegen inserting into runtime filters .. IMPALA-3360: Codegen inserting into runtime filters

[native-toolchain-CR] Bump Kudu version to 3f49724

2017-09-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/8028 Change subject: Bump Kudu version to 3f49724 .. Bump Kudu version to 3f49724 Change-Id:

[Impala-ASF-CR] Bump Kudu version to a71ecfd

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

[native-toolchain-CR] Bump Kudu version to a71ecfd

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

  1   2   3   4   5   6   >