Re: jenkins.impala.io maintenance

2017-11-08 Thread Thomas Tauber-Marshall
Jenkins has been updated. On Wed, Nov 8, 2017 at 3:13 PM Thomas Tauber-Marshall < tmarsh...@cloudera.com> wrote: > jenkins.impala.io need updates for some plugins to address a new security > advisory. > > It will be put into maintenance mode at 3:00pm PST so no new jobs can b

jenkins.impala.io maintenance

2017-11-08 Thread Thomas Tauber-Marshall
jenkins.impala.io need updates for some plugins to address a new security advisory. It will be put into maintenance mode at 3:00pm PST so no new jobs can be submitted. The upgrade will happen after all pending jobs complete. Once the upgrade completes, another email will be sent out. Please speak

Re: [VOTE] Graduate to a TLP

2017-10-17 Thread Thomas Tauber-Marshall
+1 On Tue, Oct 17, 2017 at 9:12 PM Bharath Vissapragada wrote: > +1 > > On Tue, Oct 17, 2017 at 7:10 PM, Mostafa Mokhtar > wrote: > > > +1 > > > > Thanks > > Mostafa > > > > > On Oct 17, 2017, at 7:09 PM, Brock Noland wrote: > > > > > > +1 > > > > > >> On Tue, Oct 17, 2017 at 9:07 PM, Lars Vol

Re: Broken build from Sentry

2017-06-21 Thread Thomas Tauber-Marshall
: > > > > > > > >> I was able to run a build with EE and FE tests with Sentry reverted > to > > > >> 5.12 > > > >> - unless there are objections I'm going to post a patch to revert > the > > > >> version bump. > &g

Re: Broken build from Sentry

2017-06-20 Thread Thomas Tauber-Marshall
y not that simple. The API change has been in Apache > >> > sentry > >> > > >> > So rolling back the API change temporarily solves the problem for > >> Cloudera, > >> > but we're going to have to deal with it at some point and get

Re: Broken build from Sentry

2017-06-19 Thread Thomas Tauber-Marshall
Sentry project be notified that they may want to roll back such a change? > It seems like an error on their part to do such a thing within a minor > version. > > On Mon, Jun 19, 2017 at 1:56 PM, Thomas Tauber-Marshall < > tmarsh...@cloudera.com> wrote: > > > I

Re: Broken build from Sentry

2017-06-19 Thread Thomas Tauber-Marshall
I'm working on getting the s3 jars updated, which presumably will fix that. The problem (to my understanding) is that the nightlies haven't passed since the change went into Sentry and so the Jenkins job that normally produces the new jars is still pulling in old bits. I've been talking with rele

Partial Sort Design

2017-06-14 Thread Thomas Tauber-Marshall
I'm going to be working on: https://issues.apache.org/jira/browse/IMPALA-5498, adding support for partial sorts to Impala. I've written up a brief design doc outlining my thoughts, if anyone would like to look over it and make suggestions: https://docs.google.com/document/d/1g4eJqBaZa8taA1cSsP-TJ-

Hive Schema Upgrade Required

2017-06-05 Thread Thomas Tauber-Marshall
I have two patches going out soon: https://gerrit.cloudera.org/#/c/7072/ https://gerrit.cloudera.org/#/c/7076/ which combined update Impala's Hive dependency to a new version with an updated schema. After pulling in these changes, you should find that the hive metastore will fail to come up. To fi

Re: about broadcast join and hash shuffle join

2017-05-05 Thread Thomas Tauber-Marshall
There's actually a review out right now for changing the default join algorithm when stats are unavailable to partitioned: https://gerrit.cloudera.org/#/c/6803/ On Fri, May 5, 2017 at 4:44 AM yu feng wrote: > Hi All: > > I find impala choose join algorithm by comparing data transmission size > b

Sort Expr Materialization

2017-03-06 Thread Thomas Tauber-Marshall
I'm working on a patch to materialize sort exprs, both for performance and to fix a crash, and because theres a number of issues to consider, I've put together a brief design doc, if anyone would like to look it over and make suggestions: https://docs.google.com/document/d/1tBTOzMqW53w1WSYMnqGGyPW

Could a commiter please gvm this

2017-02-09 Thread Thomas Tauber-Marshall
https://gerrit.cloudera.org/#/c/5917/ Thanks, Thomas

Could a committer...

2017-02-06 Thread Thomas Tauber-Marshall
carry forward the +2 and submit a gvm: https://gerrit.cloudera.org/#/c/5389/ Thanks, Thomas

Re: Preferred syntax for warning about ignored Status returns

2017-01-06 Thread Thomas Tauber-Marshall
I'd vote for option 1, given that, as you say, this is closely related to the return type, which putting the macro with the type makes clear. Also looking at buffer-pool.h in the review, it doesn't look overly noisy to me. On Fri, Jan 6, 2017 at 2:59 PM Tim Armstrong wrote: > Hi All, > I want

Re: Failed to build the Impala front-end

2016-10-04 Thread Thomas Tauber-Marshall
It looks like this might be related to a recent change that renamed the packages from com.cloudera.impala to org.apache.impala. Did you just fetch/pull in the patches for IMPALA-3786 before getting this error? If so, can you compile successfully without those changes? On Tue, Oct 4, 2016 at 4:28

Re: Can you please send the output of these commands?

2016-09-23 Thread Thomas Tauber-Marshall
thomas@thomas-desktop:~$cat /etc/*release DISTRIB_ID=Ubuntu DISTRIB_RELEASE=14.04 DISTRIB_CODENAME=trusty DISTRIB_DESCRIPTION="Ubuntu 14.04.4 LTS" NAME="Ubuntu" VERSION="14.04.4 LTS, Trusty Tahr" ID=ubuntu ID_LIKE=debian PRETTY_NAME="Ubuntu 14.04.4 LTS" VERSION_ID="14.04" HOME_URL="http://www.ubunt

Re: [Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 2)

2016-09-01 Thread Thomas Tauber-Marshall
On Thu, Sep 1, 2016 at 6:58 PM Thomas Tauber-Marshall < tmarsh...@cloudera.com> wrote: > Current status: > > I can get the packaging build as far as successfully generating the > parcels and setting up the cm cluster. Example run: > http://golden.jenkins.cloudera.com/job/CDH

Re: [Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 2)

2016-09-01 Thread Thomas Tauber-Marshall
I won't be working on this more for now. I'm not sure how urgent it is, but of course someone can take over from here if needed. On Thu, Sep 1, 2016 at 11:40 AM Thomas Tauber-Marshall (Code Review) < ger...@cloudera.org> wrote: > Thomas Tauber-Marshall has posted comments on th

[Impala-CR] IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2878: Fix Base64Decode error and remove duplicate codes. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4209 To unsubscribe, visit http

[Impala-CR] IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4212 To unsubscribe, visit

[Impala-CR] IMPALA-3881: Add DataTables 1.10.12 to www/

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3881: Add DataTables 1.10.12 to www/ .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4227 To unsubscribe, visit http

[Impala-CR] IMPALA-2347: Reuse metastore client connections in Catalog

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2347: Reuse metastore client connections in Catalog .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4229 To unsubscribe, visit http

[Impala-CR] IMPALA-3952: Clear scratch batch mem pool if Open() failed.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3952: Clear scratch batch mem pool if Open() failed. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4236 To unsubscribe, visit http

[Impala-CR] IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4235 To

[Impala-CR] IMPALA-3964: Fix crash when a count(*) is performed on a nested collection.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3964: Fix crash when a count(*) is performed on a nested collection. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4237 To unsubscribe

[Impala-CR] IMPALA-3843: Update warning for non-SSSE3 CPUs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3843: Update warning for non-SSSE3 CPUs .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4218 To unsubscribe, visit http

Sorry for the spam

2016-09-01 Thread Thomas Tauber-Marshall
For anyone wondering, I just pushed the wrong branch to gerrit, resulting in a few dozen reviews being created. I'm currently working on abandoning them all. Sorry for the spam. Thanks, Thomas

[Impala-CR] Fix stress test runner bug introduced by IMPALA-3969

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Fix stress test runner bug introduced by IMPALA-3969 .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4243 To unsubscribe, visit http

[Impala-CR] IMPALA-3992: bad shell error message when running nonexistent file

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3992: bad shell error message when running nonexistent file .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4240 To unsubscribe, visit

[Impala-CR] IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4241 To unsubscribe, visit

[Impala-CR] IMPALA-3376: Extra definition level when writing Parquet files

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4207 To unsubscribe, visit http

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4213 To unsubscribe, visit

[Impala-CR] IMPALA-3924: Ubuntu16 support

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3924: Ubuntu16 support .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4216 To unsubscribe, visit http://gerrit.cloudera.org:8080

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4211 To unsubscribe, visit

[Impala-CR] IMPALA-3611: track unused Disk IO buffer memory

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3611: track unused Disk IO buffer memory .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4208 To unsubscribe, visit http

[Impala-CR] IMPALA-3790: Fix admission control flaky stress test

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3790: Fix admission control flaky stress test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4210 To unsubscribe, visit http

[Impala-CR] IMPALA-3946: fix MemPool integrity issues with empty chunks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3946: fix MemPool integrity issues with empty chunks .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4222 To unsubscribe, visit http

[Impala-CR] IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4224 To unsubscribe, visit

[Impala-CR] IMPALA-2700: ASCII NUL characters are doubled on insert into text tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2700: ASCII NUL characters are doubled on insert into text tables .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4215 To unsubscribe

[Impala-CR] IMPALA-1112: Remove some unncessary code from cross-compilation

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-1112: Remove some unncessary code from cross-compilation .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4217 To unsubscribe, visit http

[Impala-CR] IMPALA-3965: TSSLSocketWithWildcardSAN.py not exported as part of impala-shell build lib

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3965: TSSLSocketWithWildcardSAN.py not exported as part of impala-shell build lib .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4214

[Impala-CR] IMPALA-3786: Remove "Cloudera" from impalad webpage title

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3786: Remove "Cloudera" from impalad webpage title .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4219 To unsubscribe,

[Impala-CR] IMPALA-3957: Test failure in S3 build: TestLoadData.test load

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3957: Test failure in S3 build: TestLoadData.test_load .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4221 To unsubscribe, visit http

[Impala-CR] IMPALA-3940: Fix getting column stats through views.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3940: Fix getting column stats through views. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4223 To unsubscribe, visit http

[Impala-CR] IMPALA-3201: buffer pool header only

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3201: buffer pool header only .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4233 To unsubscribe, visit http://gerrit.cloudera.org:8080

[Impala-CR] IMPALA-3923: fix overflow in BufferedTupleStream::GetRows()

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3923: fix overflow in BufferedTupleStream::GetRows() .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4220 To unsubscribe, visit http

[Impala-CR] Remove dead and untested code

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Remove dead and untested code .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4232 To unsubscribe, visit http://gerrit.cloudera.org:8080

[Impala-CR] IMPALA-3972: Improve display of /varz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3972: Improve display of /varz page .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4228 To unsubscribe, visit http://gerrit.cloudera.org

[Impala-CR] IMPALA-3969: stress test: add option to set common query options

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3969: stress test: add option to set common query options .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4225 To unsubscribe, visit http

[Impala-CR] IMPALA-3829: OpenSession() logs errors on valid configuration keys

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3829: OpenSession() logs errors on valid configuration keys .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4226 To unsubscribe, visit

[Impala-CR] IMPALA-3936: BufferedBlockMgr fixes for Pin() while write in flight.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3936: BufferedBlockMgr fixes for Pin() while write in flight. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4230 To unsubscribe, visit

[Impala-CR] IMPALA-3953: Fixes for KuduScanNode BE test failure

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3953: Fixes for KuduScanNode BE test failure .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4231 To unsubscribe, visit http

[Impala-CR] IMPALA-3470: DecompressorTest is flaky.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3470: DecompressorTest is flaky. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4234 To unsubscribe, visit http://gerrit.cloudera.org

[Impala-CR] IMPALA-3063: Separate join inversion from join ordering.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3063: Separate join inversion from join ordering. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4239 To unsubscribe, visit http

[Impala-CR] IMPALA-3918: remove Cloudera copyright from the shell welcome message

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3918: remove Cloudera copyright from the shell welcome message .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4238 To unsubscribe

[Impala-CR] IMPALA-3954: Add unique database to scanner test

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3954: Add unique_database to scanner test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4242 To unsubscribe, visit http

[Impala-CR] IMPALA-3650: DISTRIBUTE BY required for managed Kudu tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3650: DISTRIBUTE BY required for managed Kudu tables .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4244 To unsubscribe, visit http

[Impala-CR] IMPALA-3090: always log memory limit errors

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3090: always log memory limit errors .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4245 To unsubscribe, visit http

[Impala-CR] Add functional and targeted perf tests for joins with empty builds

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Add functional and targeted perf tests for joins with empty builds .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4247 To unsubscribe, visit

[Impala-CR] IMPALA-3828: Enable inversion for inner joins.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3828: Enable inversion for inner joins. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4246 To unsubscribe, visit http

[Impala-CR] IMPALA-3962: Clean up scratch tuple batch on scan failures

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3962: Clean up scratch tuple batch on scan failures .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4248 To unsubscribe, visit http

[Impala-CR] IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4249 To unsubscribe, visit

[Impala-CR] IMPALA-3662: Don't double allocate tuples buffer in parquet scanner

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3662: Don't double allocate tuples buffer in parquet scanner .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4250 To unsubscribe,

[Impala-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4251 To unsubscribe

[Impala-CR] IMPALA-3996: Migrate to updated Kudu insert string API

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3996: Migrate to updated Kudu insert string API .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4252 To unsubscribe, visit http

[Impala-CR] IMPALA-4004: Don't access nested types in test failpoints.py

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-4004: Don't access nested types in test_failpoints.py .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4253 To unsubscribe, visit

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3988: Only use first 96 bits of query id .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4254 To unsubscribe, visit http

[Impala-CR] IMPALA-3090: always log memory limit errors

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
errit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3965: TSSLSocketWithWildcardSAN.py not exported as part of impala-shell build lib

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ge-Id: I5a7ab8c20c0b20c21b7f8d008e39c940419e3c4d Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil

[Impala-CR] IMPALA-3470: DecompressorTest is flaky.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7ebaa403abf45e31f38d6cf4e557d6274d877a8a Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3662: Don't double allocate tuples buffer in parquet scanner

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: anujphadke

[Impala-CR] Add functional and targeted perf tests for joins with empty builds

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
et: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3940: Fix getting column stats through views.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
errit-Change-Id: Ie3b62a5e7e7d0e84850749108c13991647cedce6 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil

[Impala-CR] IMPALA-3954: Add unique database to scanner test

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
errit-Change-Id: Ic48a4bac3df6a40cb5cb10c6f1c42583952c6c86 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Michael Brown Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
http://gerrit.cloudera.org:8080/4249 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I2546bc68ba136b2713d744c1b920878606a2217b Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer

[Impala-CR] IMPALA-3957: Test failure in S3 build: TestLoadData.test load

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
it-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil

[Impala-CR] IMPALA-3843: Update warning for non-SSSE3 CPUs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ge-Id: Id242b310715c7abea091a35e6a89647bded26c4b Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: Tomas Gavenciak

[Impala-CR] IMPALA-3964: Fix crash when a count(*) is performed on a nested collection.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ala refs/changes/37/4237/1 -- To view, visit http://gerrit.cloudera.org:8080/4237 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id0955c85d18dfba4bd29a35ec95d0355da050607 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master

[Impala-CR] IMPALA-3790: Fix admission control flaky stress test

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
oject: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong

[Impala-CR] IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
eType: newchange Gerrit-Change-Id: I9d7608181eeacfe706a09c1e153d0a3e1ee9b475 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Marcel Kornacker Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong

[Impala-CR] IMPALA-3996: Migrate to updated Kudu insert string API

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
errit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3650: DISTRIBUTE BY required for managed Kudu tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb15110b10b28ef6dd8ec136c2522b5f44dca43e Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew J

[Impala-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
-Change-Id: If59b10f20044d1a468f27810a3029fe18fb19f29 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Kathy Sun Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3972: Improve display of /varz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
701d2 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Henry Robinson Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil

[Impala-CR] Fix stress test runner bug introduced by IMPALA-3969

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
errit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Brown Gerrit-Reviewer: Tim Armstrong Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3992: bad shell error message when running nonexistent file

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
http://gerrit.cloudera.org:8080/4240 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iebcf1dc8a1a08b400a2c769a9cff38ea02c8e525 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Henry Rob

[Impala-CR] Remove dead and untested code

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
eType: newchange Gerrit-Change-Id: I49c27cbfef03ef97befa9a607b3d8d7ac6e22a43 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong

[Impala-CR] IMPALA-4004: Don't access nested types in test failpoints.py

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ewchange Gerrit-Change-Id: Idc5e67b9748a13fcd76ea5fe140e2e6b18e809b7 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
54 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia56a03ed9a1d7e77c72b66a01cd48c5b6bf3624f Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Marcel Kornacker G

[Impala-CR] IMPALA-3828: Enable inversion for inner joins.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
visit http://gerrit.cloudera.org:8080/4246 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I8ba903f1df2446350cca7e71fdb13f550bf9de72 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall G

[Impala-CR] IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
cribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6eb9a4472a65cf68edb0323b13d745277ead2e1d Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Int

[Impala-CR] IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ge-Id: Iae7612433a2e27f8887abe6624f9ee0f4867b934 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3952: Clear scratch batch mem pool if Open() failed.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
er Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3923: fix overflow in BufferedTupleStream::GetRows()

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
220/1 -- To view, visit http://gerrit.cloudera.org:8080/4220 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iabb977014d855d6edb035bf2e9519ac8b3234767 Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas T

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4211 to review the following change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header ...

[Impala-CR] IMPALA-3962: Clean up scratch tuple batch on scan failures

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
rit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3969: stress test: add option to set common query options

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Project: Impala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: David Knupp Gerrit-Reviewer: Michael Brown Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong

[Impala-CR] IMPALA-3063: Separate join inversion from join ordering.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
mpala Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: anujphadke

[Impala-CR] IMPALA-3953: Fixes for KuduScanNode BE test failure

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
er Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Sailesh Mukil Gerrit-Reviewer: Tim Armstrong

[Impala-CR] IMPALA-3376: Extra definition level when writing Parquet files

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
ol, which reads parquet files and performs minimal sanity checking on their metadata, to check for extra definition levels, and adds a test that runs the tool automatically. Change-Id: I20f25a90aa1ef74b4f00f38f832bc1c1853342c6 Reviewed-on: http://gerrit.cloudera.org:8080/3835 Reviewed-by: Thomas

  1   2   3   4   5   >