Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
Hi Bharath, Did you mean to merge this? Or did I misinterpret the Gerrit mail? Cheers, Lars On Aug 2, 2016 12:55, "Bharath Vissapragada (Code Review)" < ger...@cloudera.org> wrote: > Bharath Vissapragada has submitted this change and it was merged. > > Change subject: IMPALA-2979: Fix

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has submitted this change and it was merged. Change subject: IMPALA-2979: Fix scheduling on remote hosts .. IMPALA-2979: Fix scheduling on remote hosts Also fixes: IMPALA-2400, IMPALA-3043 This change

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada
Oops. I was sync'ing gerrit/cdh5-trunk and asf/master. They became off-sync after the move to asf. Shouldn't this be cherry-picked ? On Tue, Aug 2, 2016 at 5:27 PM, Lars Volker wrote: > Hi Bharath, > > Did you mean to merge this? Or did I misinterpret the Gerrit mail? > >

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada (Code Review)
Hello Marcel Kornacker, Lars Volker, Internal Jenkins, Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2200 to look at the new patch set (#33). Change subject: IMPALA-2979: Fix scheduling on remote hosts

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada
This commit has been cherry-picked like any other commit [1]. I pushed it to gerrit bypassing the code review. Per my understanding, gerrit checks the "Change-Id" field and sees if there is an existing review for that id and branch and sends out a message. Once it is committed to the gerrit

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
I'm not sure at all what's supposed to happen with outdated changes in cdh5-trunk. Probably this one should have been transported just as any other change merged into asf/master. Maybe your transport did the right thing and Gerrit picked it up wrongly? Did you sync git repositories or Gerrit

[Impala-CR](cdh5-trunk) IMPALA-3153: Incorrect behaviour around slash escaping single quotes

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3153: Incorrect behaviour around slash escaping single quotes .. Patch Set 1: Please add tests. Also, as a reminder, the gerrit "Impala" project is now

[Impala-CR](cdh5-trunk) IMPALA-3678: Impala FE SortNode analyzer conjuncts failure

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3678: Impala FE SortNode analyzer conjuncts_ failure .. Patch Set 1: As a reminder, the gerrit "Impala" project is now deprecated. To lear now to switch,

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 5: -Code-Review -- To view, visit http://gerrit.cloudera.org:8080/3833 To

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found This adds a test

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 2: Code-Review+2 Rebase. Reran the parquet fuzz test to validate the previous

[Impala-CR](cdh5-trunk) IMPALA-3376: Extra definition level when writing Parquet files

2016-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. Patch Set 7: Code-Review+2 (2 comments)

Re: Re: Re: IMPALA-2428 Support multiple-character string as the field delimiter

2016-08-02 Thread Jim Apple
Also, you asked > I'm wondering whether have you ever test these two cases. I do not know. Can you check and report back what you find? On Tue, Aug 2, 2016 at 3:49 PM, Jim Apple wrote: >> What's more, in this patch, we can use only standard ASCII characters(with >>

[Impala-CR](cdh5-trunk) IMPALA-2033: Netezza compatibility functions quote ident

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2033: Netezza compatibility functions quote_ident .. Patch Set 4: As a reminder, the gerrit "Impala" project is now deprecated. To lear now to switch, read

Podling Report Reminder - August 2016

2016-08-02 Thread johndament
Dear podling, This email was sent by an automated system on behalf of the Apache Incubator PMC. It is an initial reminder to give you plenty of time to prepare your quarterly board report. The board meeting is scheduled for Wed, 17 August 2016, 10:30 am PDT. The report for your podling will form

Re: Re: Re: IMPALA-2428 Support multiple-character string as the field delimiter

2016-08-02 Thread Jim Apple
> What's more, in this patch, we can use only standard ASCII characters(with > decimal value from 0 to 127) in ascii or octal format to set filed > terminator, but not extended ASCII characters(with decimal value from 128 to > 255) or standard ASCII characters in unicode, decimal or hexadecimal

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

2016-08-02 Thread Tim Armstrong (Code Review)
Hello Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3833 to look at the new patch set (#4). Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 4: Code-Review+2 Didn't bring across the new files when cherry-picking to new

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

2016-08-02 Thread Tim Armstrong (Code Review)
Hello Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3833 to look at the new patch set (#5). Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 3: Code-Review+2 Missed porting a couple of range checks during rebase. --

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found This adds a test

[Impala-CR](cdh5-trunk) IMPALA-2328 Parquet scan should use min/max stats

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2328 Parquet scan should use min/max stats .. Patch Set 1: As a reminder, the gerrit "Impala" project is now deprecated. To learn now to switch, read

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

2016-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. Patch Set 1: Code-Review+1 Replaces https://gerrit.cloudera.org/#/c/3556/ -- To

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 5: Code-Review-2 Running the fuzzer found another crash in parquet that I

[Impala-CR](cdh5-trunk) IMPALA-1654: Support general predicates in most partition DDL operations.

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-1654: Support general predicates in most partition DDL operations. .. Patch Set 9: As a reminder, the gerrit "Impala" project is now deprecated. To lear now

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

2016-08-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 3: Verified-1 Build failed:

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

2016-08-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 5: Verified-1 Build failed:

[Impala-CR](cdh5-trunk) IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables

2016-08-02 Thread Feng Guangyuan (Code Review)
Feng Guangyuan has restored this change. Change subject: IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables .. Restored -- To view, visit http://gerrit.cloudera.org:8080/3266 To unsubscribe, visit

[Impala-CR](cdh5-trunk) IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables

2016-08-02 Thread Feng Guangyuan (Code Review)
Feng Guangyuan has posted comments on this change. Change subject: IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables .. Patch Set 2: (3 comments) Yes Jim, I will end it. Thanks for your patient,

[Impala-CR](cdh5-trunk) IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables

2016-08-02 Thread Feng Guangyuan (Code Review)
Feng Guangyuan has uploaded a new patch set (#2). Change subject: IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables .. IMPALA-3622: Unacceptable '\0' used as fields terminator when creating tables

Re: Fw: Issues with tests in Release-mode Impala build

2016-08-02 Thread Valencia Serrao
Hi Tim, To trace the exploration strategy var at each step leading to the "test_spilling" test, I put a few print statements for it. I observed that, once the process flow reaches the impala_test_suite.py the strategy is changed to 'core'. The logs printed as follows: (logs marked pink)

Re: Re: Re: IMPALA-2428 Support multiple-character string as the field delimiter

2016-08-02 Thread Yuanhao Luo
Hello, Jim Apple. For now in my commit, field terminators can not be set to extended ASCIII characters. After using statement "create table text_thorn_ecirc_newline(col1 string, col2 string, col3 int, col4 int) row format delimited fields terminated by 'þ' escaped by '-22' lines termiated by

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

2016-08-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3835 To

[Impala-CR](cdh5-trunk) IMPALA-1766: Misc. statistical functions. Implemented aggregate corr().

2016-08-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-1766: Misc. statistical functions. Implemented aggregate corr(). .. Patch Set 2: As a reminder, the gerrit "Impala" project is now deprecated. To lear now

[Impala-CR](cdh5-trunk) IMPALA-3376: Extra definition level when writing Parquet files

2016-08-02 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 Replaced with: https://gerrit.cloudera.org/#/c/3835/ for Impala-ASF -- To view,

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

2016-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/3835 Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. IMPALA-3376: Extra definition level when

Re: Podling Report Reminder - August 2016

2016-08-02 Thread Jim Apple
bcc: dev@ Hi John, This is the third reminder we have received to dev@. The report is uploaded and one of our mentors has already signed off. Is there some other thing that needs to be done? Thanks, Jim On Tue, Aug 2, 2016 at 5:07 PM, wrote: > Dear podling, > > This

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
Just to clarify, this change has been moved to Impala-ASF and merged successfully. On Aug 2, 2016 18:30, "Jim Apple" wrote: > For outdated changes, please use the instructions here: > > >

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Jim Apple
For outdated changes, please use the instructions here: http://www.google.com/url?q=http%3A%2F%2Fmail-archives.apache.org%2Fmod_mbox%2Fincubator-impala-dev%2F201607.mbox%2F%253CCAC-pSX2sHyqsoxGCCKrqmQbo-jjWdP6CMGoTU_1snuQVkDVg_w%40mail.gmail.com%253E=D=1=AFQjCNFyYa3BsqALZuEvxDzgAV5l4P_evQ "For

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Jim Apple
Please do not commit to the "Impala" gerrit project any more. It is stale and it will remain stale. Use "Impala-ASF" for asf and use private jenkins for private github. Public github (corresponding to public gerrit project "Impala") should be stale at the moment. Please see our branching document

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has abandoned this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Abandoned Moved to https://gerrit.cloudera.org/#/c/3833/1 -- To view, visit

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/3833 Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. IMPALA-3764,3914: fuzz test HDFS scanners and

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Patch Set 1: Code-Review+2 Moved from https://gerrit.cloudera.org/#/c/3448/9 Carry +2

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

2016-08-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3611: track unused Disk IO buffer memory .. Patch Set 4: Fixed another couple of problems and ran a bunch more tests. Sorry for the churn, but it should

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

2016-08-02 Thread Tim Armstrong (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3799 to look at the new patch set (#4). Change subject: IMPALA-3611: track unused Disk IO buffer memory