[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created

2017-05-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created .. Patch Set 3: (8 comments) http://gerrit.cloudera.org:8080/#/c/6792/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5120: Default to partitioned join when stats are missing

2017-05-08 Thread Thomas Tauber-Marshall (Code Review)
: newpatchset Gerrit-Change-Id: Ie168ecfcd5e7c5d3c60d16926c151f8f134c81e0 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Marcel Ko

[Impala-ASF-CR] IMPALA-5120: Default to partitioned join when stats are missing

2017-05-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5120: Default to partitioned join when stats are missing .. Patch Set 2: Code-Review+2 Carrying forward -- To view, visit http

[native-toolchain-CR] Bump Kudu version to 7533364.

2017-05-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 7533364. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6796/1//COMMIT_MSG Commit Message: PS1, Line 7: . nit

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

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

[native-toolchain-CR] Bump Kudu version to 7533364.

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

[Impala-ASF-CR] IMPALA-5120: Default to partitioned join when stats are missing

2017-05-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6803 Change subject: IMPALA-5120: Default to partitioned join when stats are missing .. IMPALA-5120: Default to partitioned join

[Impala-ASF-CR] IMPALA-5167: Use a single Kudu client across the BE

2017-05-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6792 Change subject: IMPALA-5167: Use a single Kudu client across the BE .. IMPALA-5167: Use a single Kudu client across

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-05-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6790 Change subject: IMPALA-5080: OutOfMemory PermGen space .. IMPALA-5080: OutOfMemory PermGen space A previous fix

[Impala-ASF-CR] IMPALA-5137: Support Kudu UNIXTIME MICROS as Impala TIMESTAMP

2017-05-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5137: Support Kudu UNIXTIME_MICROS as Impala TIMESTAMP .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/6526/5//COMMIT_MSG

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6775/1/tests/query_test

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. IMPALA-5262: test_analytic_order_by_random fails with assert This was a poorly written

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6775/1/tests/query_test

[Impala-ASF-CR] IMPALA-5262: test analytic order by random fails with assert

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6775 Change subject: IMPALA-5262: test_analytic_order_by_random fails with assert .. IMPALA-5262: test_analytic_order_by_random

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6559

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
-Change-Id: I84ce0032a1b10958fdf31faef225372c5c38fdc4 Gerrit-PatchSet: 9 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Dimitris Tsirogiannis

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 8: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/6559/7/fe/src

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-05-01 Thread Thomas Tauber-Marshall (Code Review)
-MessageType: newpatchset Gerrit-Change-Id: I84ce0032a1b10958fdf31faef225372c5c38fdc4 Gerrit-PatchSet: 8 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Dimitris

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#7). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-27 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#6). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] Marcel's review 2.0

2017-04-27 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6753 Change subject: Marcel's review 2.0 .. Marcel's review 2.0 Change-Id: Ieb788780ad570b84d5660a6f6ca0c9c24f3eb016 --- M be/src

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-26 Thread Thomas Tauber-Marshall (Code Review)
-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 9: Code-Review+2 Rebased. -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-26 Thread Thomas Tauber-Marshall (Code Review)
://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifefdaff8557a30ac44ea82ed428e6d1ffbca2e9e Gerrit-PatchSet: 8 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@clo

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 5: (24 comments) http://gerrit.cloudera.org:8080/#/c/6559/4/common/thrift

[Impala-ASF-CR] Bump Kudu version to 238249c

2017-04-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6718 Change subject: Bump Kudu version to 238249c .. Bump Kudu version to 238249c This will pull in the Kudu client partitioner API

[Impala-ASF-CR] IMPALA-5152: Gather all tables with missing metadata in analysis

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5152: Gather all tables with missing metadata in analysis .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6702/1/fe/src/main

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 238249c. .. Patch Set 1: This passed a toolchain verify build: http://unittest.jenkins.cloudera.com/job/verify-impala-toolchain

[native-toolchain-CR] Bumped Kudu version to 238249c.

2017-04-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6711 Change subject: Bumped Kudu version to 238249c. .. Bumped Kudu version to 238249c. This will pull in the Kudu client

[Impala-ASF-CR] IMPALA-5217: KuduTableSink checks null constraints incorrectly

2017-04-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5217: KuduTableSink checks null constraints incorrectly .. Patch Set 2: GVO failure definitely unrelated (its an HDFS test, but this patch only

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 4: As requested, I ran some tests with tpcds_1000_text.store_sales: with patch

[Impala-ASF-CR] IMPALA-5217: KuduTableSink checks null constraints incorrectly

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5217: KuduTableSink checks null constraints incorrectly .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6670/1

[Impala-ASF-CR] IMPALA-5217: KuduTableSink checks null constraints incorrectly

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com>

[Impala-ASF-CR] IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
: newpatchset Gerrit-Change-Id: Ic20b1621753980b47a612e0885804363b733f6da Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>

[Impala-ASF-CR] IMPALA-5217: KuduTableSink checks null constraints incorrectly

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6670 Change subject: IMPALA-5217: KuduTableSink checks null constraints incorrectly .. IMPALA-5217: KuduTableSink checks null

[Impala-ASF-CR] IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/6653/1

[Impala-ASF-CR] IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates .. IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates This patch

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 7: Rebased -- To view, visit http://gerrit.cloudera.org:8080/6322

[Impala-ASF-CR] IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates

2017-04-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6653 Change subject: IMPALA-5125: SimplifyConditionalsRule incorrectly handles aggregates .. IMPALA-5125: SimplifyConditionalsRule

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5080: OutOfMemory PermGen space .. Patch Set 3: Code-Review+2 Carrying forward Alex's +2 -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5080: OutOfMemory PermGen space .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6642/2/bin/impala-config.sh File bin/impala

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-17 Thread Thomas Tauber-Marshall (Code Review)
/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iccfb69cdf7958e053890089db6a82ad491fb5b9d Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Ger

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5080: OutOfMemory PermGen space .. Patch Set 2: (1 comment) > (1 comment) > > Have you tried Bharath's URLClassLoader.close() s

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-14 Thread Thomas Tauber-Marshall (Code Review)
Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iccfb69cdf7958e053890089db6a82ad491fb5b9d Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Revie

[Impala-ASF-CR] IMPALA-5080: OutOfMemory PermGen space

2017-04-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6642 Change subject: IMPALA-5080: OutOfMemory PermGen space .. IMPALA-5080: OutOfMemory PermGen space The problem is that over time

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 4: Perf results from running on the 10 node cluster: For smaller queries that we

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-13 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 7: Code-Review+2 (2 comments) Carrying forward Alex's +2 http

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-11 Thread Thomas Tauber-Marshall (Code Review)
-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Marcel Kornacker <mar...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 5: (16 comments) http://gerrit.cloudera.org:8080/#/c/6322/4/fe/src/main/java

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#6). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 5: Sorry, didn't mean to resubmit. Please ignore patch set 5. -- To view

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Abandoned Submitted by mistake. -- To view, visit http://gerrit.cloudera.org:8080/6593

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. Patch Set 2: (25 comments) http://gerrit.cloudera.org:8080/#/c/6559/1/be/src/exec/kudu

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6593 Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 3: (13 comments) http://gerrit.cloudera.org:8080/#/c/6322/1/fe/src/main/java

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-04-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-04-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Abandoned Replaced by https://gerrit.cloudera.org/#/c/6559/ -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-3742: Partitions and sort INSERTs for Kudu tables

2017-04-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6559 Change subject: IMPALA-3742: Partitions and sort INSERTs for Kudu tables .. IMPALA-3742: Partitions and sort INSERTs for Kudu

[Impala-ASF-CR] IMPALA-4166: Add SORT BY sql clause

2017-03-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4166: Add SORT BY sql clause .. Patch Set 3: (6 comments) Just a heads up, this is going to have some conflicts with IMPALA-3742 (partition

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 10: Sorry, didn't mean to update this, not ready for re-review. -- To view, visit http

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#10). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#10). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#9). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 9: (4 comments) http://gerrit.cloudera.org:8080/#/c/6037/8/be/src/runtime/data-stream

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6037/8/be/src/runtime/data-stream

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 8: (4 comments) http://gerrit.cloudera.org:8080/#/c/6037/7/be/src/runtime/data-stream

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#8). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 2: (24 comments) http://gerrit.cloudera.org:8080/#/c/6322/1/fe/src/main/java

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-13 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 7: (14 comments) http://gerrit.cloudera.org:8080/#/c/6037/6/be/src/runtime/data-stream

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-13 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#7). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-4318: Kudu support for CREATE EXTERNAL TABLE AS SELECT

2017-03-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4318: Kudu support for CREATE EXTERNAL TABLE AS SELECT .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6261/4//COMMIT_MSG

[Impala-ASF-CR] IMPALA-4996: Single-threaded KuduScanNode

2017-03-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4996: Single-threaded KuduScanNode .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6312/1/be/src/exec/kudu-scan-node-base.h

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#6). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6322 Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. Patch Set 5: (14 comments) Still working on performance. http://gerrit.cloudera.org:8080/#/c/6037/4

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-3742: partitions DMLs for Kudu tables

2017-03-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-3742: partitions DMLs for Kudu tables .. IMPALA-3742: partitions DMLs for Kudu tables Bulk DMLs (INSERT, UPSERT, UPDATE, and DELETE) for Kudu

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Abandoned Still working out the design. Doc: https://docs.google.com/document/d

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. Patch Set 3: (1 comment) A couple of notes on the latest review: - It has not been

[Impala-ASF-CR] IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs

2017-03-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs .. IMPALA-4731/IMPALA-397/IMPALA-4728: Materialize sort exprs Previously, exprs used in sorts

[Impala-ASF-CR] PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables

2017-03-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/6037/1/be/src/runtime/data

[Impala-ASF-CR] PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables

2017-03-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables .. PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables Bulk inserts into Kudu

[Impala-ASF-CR] PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables

2017-02-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables .. PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables Bulk inserts into Kudu

[Impala-ASF-CR] PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables

2017-02-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6037 Change subject: PREVIEW: IMPALA-3742: partitions INSERTs into Kudu tables .. PREVIEW: IMPALA-3742: partitions INSERTs into Kudu

[Impala-ASF-CR] IMPALA-4828: Alter Kudu schema outside Impala may crash on read

2017-02-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4828: Alter Kudu schema outside Impala may crash on read .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5840

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5941/1/testdata/workloads/tpch

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I4731a3e83dd2142a1d83be963f83cd1847472295 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com> Gerrit-R

[Impala-ASF-CR] IMPALA-4895: Memory limit exceeded in test outer joins

2017-02-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/5941 Change subject: IMPALA-4895: Memory limit exceeded in test_outer_joins .. IMPALA-4895: Memory limit exceeded

[Impala-ASF-CR] IMPALA-4849: IllegalStateException from rewritten CASE expr

2017-02-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4849: IllegalStateException from rewritten CASE expr .. IMPALA-4849: IllegalStateException from rewritten CASE expr In SelectList.reset(), we call

[Impala-ASF-CR] IMPALA-4731: Crash when sorting on non-deterministic expr

2017-02-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4731: Crash when sorting on non-deterministic expr .. IMPALA-4731: Crash when sorting on non-deterministic expr When sorting on a non

[Impala-ASF-CR] IMPALA-4828: Alter Kudu schema outside Impala may crash on read

2017-02-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4828: Alter Kudu schema outside Impala may crash on read .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5840/2/be/src/exec

<    1   2   3   4   5   6   >