[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 8: Code-Review+2 Verified+1 Carry +2 https://jenkins.impala.io/job/parallel-all-tests/1284

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has submitted this change and it was merged. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. IMPALA-5412: Fix scan result with partitions on same file The maps storing file descriptors

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 7: Verified+1 https://jenkins.impala.io/job/parallel-all-tests/1284/ -- To view, visit h

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 7: Running tests against the Impala-lzo branch with matching change: https://jenkins.impa

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 7: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/104

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1040/ -- To

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 7: Code-Review+2 Rebase -- To view, visit http://gerrit.cloudera.org:8080/7625 To unsubs

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7625 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7625/5/be/src/exec/hdfs-scan-node-bas

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7625 to look at the new patch set (#6). Change subject: IMPALA-5412: Fix scan result with partitions on same file .. I

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/7625/5/be/src/exec/hdfs-scan-node-base.h

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5412: Fix scan result with partitions on same file .. Patch Set 5: Code-Review+1 Looks good -- To view, visit http://gerrit.cloudera.org:8080/7625 To u

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412 Fix scan result with partitions on same file .. Patch Set 3: (3 comments) I ended up combining the tests in test_partition_metadata and adding parqu

[Impala-ASF-CR] IMPALA-5412: Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-5412: Fix scan result with partitions on same file .. IMPALA-5412: Fix scan result with partitions on same file The maps storing file descriptors and file m

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-5412 Fix scan result with partitions on same file .. IMPALA-5412 Fix scan result with partitions on same file The maps storing file descriptors and file met

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-10 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5412 Fix scan result with partitions on same file .. Patch Set 3: (1 comment) > (2 comments) > > Gabor told me that he won't be able to work on this

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5412 Fix scan result with partitions on same file .. Patch Set 3: (2 comments) Gabor told me that he won't be able to work on this for around 10 days. Fr

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. Change subject: IMPALA-5412 Fix scan result with partitions on same file .. Patch Set 1: (20 comments) http://gerrit.cloudera.org:8080/#/c/7625/1/be/src/exec/base-sequence-scann

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#3). Change subject: IMPALA-5412 Fix scan result with partitions on same file .. IMPALA-5412 Fix scan result with partitions on same file The maps storing file descriptors and file meta

[Impala-ASF-CR] IMPALA-5412 Fix scan result with partitions on same file

2017-08-10 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has uploaded a new patch set (#2). Change subject: IMPALA-5412 Fix scan result with partitions on same file .. IMPALA-5412 Fix scan result with partitions on same file The maps storing file descriptors and file meta