[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. IMPALA-2020, 4915, 4936: Add rounding for decimal casts This change adds support for

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 23: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5951 To unsubscrib

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 4: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/301/

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 4: Code-Review+2 rebase, carry +2 -- To view, visit http://gerrit.cloudera.org:8080/6127

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6127 To un

[Impala-ASF-CR] IMPALA-3405: [DOCS] Pare down upgrading section

2017-02-23 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: IMPALA-3405: [DOCS] Pare down upgrading section .. Patch Set 1: There is a merge conflict and after resolving it, the rendered file includes the full last line "T

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6127 to look at the new patch set (#3). Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). ...

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6127/2/fe/src/main/java/org/apache/im

[Impala-ASF-CR] IMPALA-3403: [DOCS] Pare back irrelevant installation info

2017-02-23 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: IMPALA-3403: [DOCS] Pare back irrelevant installation info .. Patch Set 2: Code-Review+1 (1 comment) Changes build cleanly and topic looks good. http://gerrit.cl

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 5: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/298/ -- To view, vi

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6127/2/fe/src/main/java/o

[Impala-ASF-CR] IMPALA-4955: Fix integer overflow in hdfs table size accounting

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4955: Fix integer overflow in hdfs table size accounting .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/300

[Impala-ASF-CR] IMPALA-4955: Fix integer overflow in hdfs table size accounting

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4955: Fix integer overflow in hdfs table size accounting .. Patch Set 1: Code-Review+2 Actually, I'll give +2. It's a very simple fix. -- To view, visit htt

[Impala-ASF-CR] IMPALA-4955: Fix integer overflow in hdfs table size accounting

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4955: Fix integer overflow in hdfs table size accounting .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6133 To unsubscribe,

[Impala-ASF-CR] IMPALA-4955: Fix integer overflow in hdfs table size accounting

2017-02-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/6133 Change subject: IMPALA-4955: Fix integer overflow in hdfs table size accounting .. IMPALA-4955: Fix integer overflow in hdfs table

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6127/1/fe/src/main/java/org

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. IMPALA-4981: Re-enable spilling with MT_DOP. The initial changes for MT_DOP disabled spilling be

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6131 To unsubscribe, visit htt

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#9). Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. IMPALA-4787: Optimize APPX_MEDIAN() memory usage Before this change, ReservoirSample functions (such as APP

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Hello Matthew Jacobs, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6025 to look at the new patch set (#9). Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage ..

[Impala-ASF-CR] IMPALA-4787: Optimize APPX MEDIAN() memory usage

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4787: Optimize APPX_MEDIAN() memory usage .. Patch Set 8: (11 comments) http://gerrit.cloudera.org:8080/#/c/6025/8//COMMIT_MSG Commit Message: PS8,

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 23: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/299/ --

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 23: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5951 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4848: Add WIDHT BUCKET() function

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4848: Add WIDHT_BUCKET() function .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6023/3/be/src/exprs/math-functions-ir.cc File be/src/exprs/

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 22: (1 comment) http://gerrit.cloudera.org:8080/#/c/5951/22/be/src/runtime/decimal-value.inlin

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6127/1/fe/src/main/java/org/apache/imp

[Impala-ASF-CR] IMPALA-4624: Implement Parquet dictionary filtering

2017-02-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#12). Change subject: IMPALA-4624: Implement Parquet dictionary filtering .. IMPALA-4624: Implement Parquet dictionary filtering Here is a basic summary of the changes: Frontend looks

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/11/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4848: Add WIDHT BUCKET() function

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4848: Add WIDHT_BUCKET() function .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/6023/3/be/src/exprs/math-functions-ir.cc File be/src/exprs

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/11/fe/src/main/java/org/apache

[Impala-ASF-CR] IMPALA-4624: Implement Parquet dictionary filtering

2017-02-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#11). Change subject: IMPALA-4624: Implement Parquet dictionary filtering .. IMPALA-4624: Implement Parquet dictionary filtering Here is a basic summary of the changes: Frontend looks

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 2: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/6127/1/fe/src

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (7 comments) http://gerrit.cloudera.org:8080/#/c/6032/11/be/src/exec/hdfs-parquet-scanner.

[Impala-ASF-CR] IMPALA-4813: Round on divide and multiply

2017-02-23 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-4813: Round on divide and multiply .. IMPALA-4813: Round on divide and multiply Address rounding on divide and multiply when results are truncated. Testing:

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 5: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/298/ -- To view, visit http:/

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6130 To unsubscribe, visit http://gerrit.clouder

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 5: (4 comments) Thank you for the review. Please see PS5. http://gerrit.cloudera.org:8080/#/c/6130/4/testdata/wo

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#5). Change subject: IMPALA-4982: Add parquet stats test .. IMPALA-4982: Add parquet stats test IMPALA-2328 added support for skipping row groups based on parquet::Statistics. This chang

[Impala-ASF-CR] IMPALA-4624: Implement Parquet dictionary filtering

2017-02-23 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4624: Implement Parquet dictionary filtering .. Patch Set 10: (2 comments) http://gerrit.cloudera.org:8080/#/c/5904/10/be/src/exec/hdfs-parquet-scanne

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6130/4/testdata/workloads/functional-query/queries/QueryTest/ne

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 3: (2 comments) Thank you for the review! Please see my comments and PS4. http://gerrit.cloudera.org:8080/#/c/61

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 22: (1 comment) http://gerrit.cloudera.org:8080/#/c/5951/22/be/src/runtime/decimal-value.inli

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#4). Change subject: IMPALA-4982: Add parquet stats test .. IMPALA-4982: Add parquet stats test IMPALA-2328 added support for skipping row groups based on parquet::Statistics. This chang

[Impala-ASF-CR] IMPALA-4624: Implement Parquet dictionary filtering

2017-02-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new patch set (#10). Change subject: IMPALA-4624: Implement Parquet dictionary filtering .. IMPALA-4624: Implement Parquet dictionary filtering Here is a basic summary of the changes: Frontend looks

[Impala-ASF-CR] IMPALA-4624: Implement Parquet dictionary filtering

2017-02-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-4624: Implement Parquet dictionary filtering .. Patch Set 9: (28 comments) http://gerrit.cloudera.org:8080/#/c/5904/9/be/src/exec/hdfs-parquet-scanner.cc

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-02-23 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 9: I'll review the be side. -- To view, visit http://gerrit.cloudera.org:8080/5816 To unsubscribe, vis

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4982: Add parquet stats test .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/6130/3/testdata/workloads/functional-query/queries/QueryTest/pa

[Impala-ASF-CR] IMPALA-4813: Round on divide and multiply

2017-02-23 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/6132 Change subject: IMPALA-4813: Round on divide and multiply .. IMPALA-4813: Round on divide and multiply Address rounding on divide and mult

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6127/1//COMMIT_MSG Commit Message: P

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). The bug: When generating the toThrift()

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6127/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/297/ -- To view, vis

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6131 To unsubscribe, visit http://gerri

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6131 To unsubscribe, visit http://gerri

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6125 To unsubsc

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. [DOCS] Construct "fixed issues" topic for 2.8 release notes Primarily a link to t

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. [DOCS] Fix build warning due to removal of FAQ entries. Change-Id: I9d4d64e73ecbad0a3

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6126 To unsubscribe

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/50/ --

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 2: Code-Review+2 +2ing based on Laurel's +1 plus Jim's comment had a straightforward fix.

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6125/1/docs/topics/impala_fixed_issu

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
Hello Laurel Hale, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6125 to look at the new patch set (#2). Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. [D

[Impala-ASF-CR] IMPALA-4981: Re-enable spilling with MT DOP.

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/6131 Change subject: IMPALA-4981: Re-enable spilling with MT_DOP. .. IMPALA-4981: Re-enable spilling with MT_DOP. The initial changes for MT_DOP

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6126 To unsubscribe, visi

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. Patch Set 1: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/49/ -- To v

[Impala-ASF-CR] [DOCS] Add several items to "known issues" page

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Add several items to "known issues" page .. Patch Set 1: Yes, still active, just a couple of other items in the queue ahead of it right now. -- To view,

[Impala-ASF-CR] IMPALA-2020, 4915, 4936: Add rounding for decimal casts

2017-02-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-2020, 4915, 4936: Add rounding for decimal casts .. Patch Set 22: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/5951/22/be/src/runtime/decim

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the u

2017-02-23 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 5 of "Cloudera Manager" removal. Most of these fixes entailed hiding the paragraph where there are mentions of CM and writing a replacement paragraph for the upstream docs. This removes the CM references f

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 9: Code-Review+1 (1 comment) FE changes lgtm. http://gerrit.cloudera.org:8080/#/c/5816/9/be/src/exec/union-no

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-4982: Add parquet stats test .. IMPALA-4982: Add parquet stats test IMPALA-2328 added support for skipping row groups based on parquet::Statistics. This chang

[Impala-ASF-CR] IMPALA-4982: Add parquet stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-4982: Add parquet stats test .. IMPALA-4982: Add parquet stats test IMPALA-2328 added support for skipping row groups based on parquet::Statistics. This chang

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#9). Change subject: IMPALA-3586: Implement union passthrough .. IMPALA-3586: Implement union passthrough The union node acts as pass through operator and forwards row batches from

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#9). Change subject: IMPALA-3586: Implement union passthrough .. IMPALA-3586: Implement union passthrough The union node acts as pass through operator and forwards row batches from

[Impala-ASF-CR] IMPALA-3586: Implement union passthrough

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3586: Implement union passthrough .. Patch Set 8: (7 comments) http://gerrit.cloudera.org:8080/#/c/5816/8/be/src/exec/union-node.h File be/src/exec/u

[Impala-ASF-CR] IMPALA-4982: Add parques stats test

2017-02-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/6130 Change subject: IMPALA-4982: Add parques stats test .. IMPALA-4982: Add parques stats test IMPALA-2328 added support for skipping row grou

[Impala-ASF-CR] Three misc webpage changes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Three misc webpage changes .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6045 To unsubscribe, visit http://gerrit.clouder

[Impala-ASF-CR] Three misc webpage changes

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: Three misc webpage changes .. Three misc webpage changes 1. Change the content width to 80% of the screen. Lots of our webpages are very wide, a

[Impala-ASF-CR] IMPALA-3401 [DOCS] Part 4 of "Cloudera Manager" removal. Most of these fixes involved hiding the paragraphs with the DITA attribute 'audience="hidden"' and then inserting a paragraph s

2017-02-23 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Part 4 of "Cloudera Manager" removal. Most of these fixes involved hiding the paragraphs with the DITA attribute 'audience="hidden"' and then inserting a paragraph suitable for upstream documentation. This hides

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/5969/6/be/src/exprs/timestamp-functions.cc F

[Impala-ASF-CR] IMPALA-3401 [DOCS] Phase 3 of removing Cloudera Manager from upstream docs.

2017-02-23 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Phase 3 of removing Cloudera Manager from upstream docs. .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/6067/2/docs/topics/

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-23 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5969/6/be/src/exprs/timestamp-functio

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 1: Code-Review+1 (1 comment) Did a build and no unexpected errors thrown. Rendered docs l

[Impala-ASF-CR] IMPALA-4546: Fix Moscow timezone conversion after 2014

2017-02-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4546: Fix Moscow timezone conversion after 2014 .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5969/6/be/src/exprs/timestamp-functions.cc Fi

[Impala-ASF-CR] IMPALA-3401 [DOCS] Removing 'Cloudera Manager' from upstream docs.

2017-02-23 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3401 [DOCS] Removing 'Cloudera Manager' from upstream docs. .. Patch Set 2: (12 comments) http://gerrit.cloudera.org:8080/#/c/6064/2/docs/topics/impala_a

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. Patch Set 1: Code-Review+1 Clean build--no error messages thrown and change in doc looks good. -- To v

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread Ambreen Kazi (Code Review)
Ambreen Kazi has posted comments on this change. Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. Patch Set 1: Code-Review+1 Local build looks good. -- To view, visit http://gerrit.cloudera.org:8080/

[Impala-ASF-CR] IMPALA-4902: Copy parameters map in HdfsPartition.toThrift().

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/6127 Change subject: IMPALA-4902: Copy parameters map in HdfsPartition.toThrift(). .. IMPALA-4902: Copy parameters map in HdfsPartition.toThrift()

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic Th

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5962

[Impala-ASF-CR] [DOCS] Fix build warning due to removal of FAQ entries.

2017-02-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6126 Change subject: [DOCS] Fix build warning due to removal of FAQ entries. .. [DOCS] Fix build warning due to removal of FAQ entries. Change

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 7: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit

[Impala-ASF-CR] IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic

2017-02-23 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic .. Patch Set 7: Code-Review+2 The link error is actually from a different gerrit (stripping

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6125/1/docs/topics/impala_fixed_issues.x

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/6125 Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. [DOCS] Construct "fixed issues" topic for 2.8 release notes

[Impala-ASF-CR] [DOCS] Construct "fixed issues" topic for 2.8 release notes

2017-02-23 Thread John Russell (Code Review)
John Russell has abandoned this change. Change subject: [DOCS] Construct "fixed issues" topic for 2.8 release notes .. Abandoned Unintended changes to new features list got mixed in. Impractical to resolve all conflicts. Going

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Release note updates for Impala 2.8 .. [DOCS] Release note updates for Impala 2.8 First cut at 'new features' topic. Includes the Incomp

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Release note updates for Impala 2.8 .. Patch Set 9: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5668 To unsubscribe, visit http:

[Impala-ASF-CR] IMPALA-4936 and IMPALA-4915: Fix decimal overflow test

2017-02-23 Thread Zach Amsden (Code Review)
Zach Amsden has abandoned this change. Change subject: IMPALA-4936 and IMPALA-4915: Fix decimal overflow test .. Abandoned Rolled this into casting change -- To view, visit http://gerrit.cloudera.org:8080/6068 To unsubscribe,

[Impala-ASF-CR] IMPALA-4936 and IMPALA-4915: Fix decimal overflow test

2017-02-23 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-4936 and IMPALA-4915: Fix decimal overflow test .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6068/2/be/src/runtime/decimal-value.inline.

[Impala-ASF-CR] [DOCS] Release note updates for Impala 2.8

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Release note updates for Impala 2.8 .. Patch Set 9: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/47/ -- To view, visit ht

  1   2   >