[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

[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

[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

[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

[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)

[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.

[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,

[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)

[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:

[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

[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

[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

[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)

[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

[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

[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

[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

[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)

[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)

[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)

[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

[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)

[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)

[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)

[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

[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

[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.

[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

[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)

[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)

[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.

[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)

[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

[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)

[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,

[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)

[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

[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:

[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,

[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

[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

[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

[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

[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:

[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

[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

[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)

[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 ..

[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,

[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

[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

[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)

[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

[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.

[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

[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

[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

[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

[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

[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

[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

[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,

[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

[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

[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)

[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)

[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

[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

[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

[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

[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

[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

[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.

[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:

[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)

[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

[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.

[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

[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

[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

[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)

[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

[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: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/296/ -- To view, visit

[Impala-ASF-CR] Three misc webpage changes

2017-02-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Three misc webpage changes .. Patch Set 1: Code-Review+2 > Thanks for the reviews! > > > seems fine but I don't really know jquery > > If it makes it

  1   2   >