[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5776/2/be/src/exprs/expr-test.cc File be/src/exprs/expr-test.cc:

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/5776/2/be/src/exprs/string-functions-ir.cc File

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-23 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#7). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] IMPALA-1654: [DOCS] DDL for multiple partitions

2017-01-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1654: [DOCS] DDL for multiple partitions .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5726 To unsubscribe, visit

[Impala-ASF-CR] Remove 'without Cloudera Manager' from two titles

2017-01-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: Remove 'without Cloudera Manager' from two titles .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5749 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 2: (10 comments) http://gerrit.cloudera.org:8080/#/c/5776/2//COMMIT_MSG Commit Message: Line 11: since function

[Impala-ASF-CR] IMPALA-3202: implement spill-to-disk in new buffer pool

2017-01-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool .. Patch Set 12: (19 comments)

[Impala-ASF-CR] IMPALA-4670 / IMPALA-4672: Add RpcMgr and port Statestore services to KRPC

2017-01-23 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-4670 / IMPALA-4672: Add RpcMgr and port Statestore services to KRPC .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-4729: Implement REPLACE() .. IMPALA-4729: Implement REPLACE() This turned out to be slightly non-trivial as REPLACE is already a keyword, and thus the parser

[Impala-ASF-CR] IMPALA-4482: Use ALTER TABLE / RECOVER PARTITIONS when loading tpcds.store sales

2017-01-23 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4482: Use ALTER TABLE / RECOVER PARTITIONS when loading tpcds.store_sales .. Patch Set 6: What's the next step? -- To view, visit

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-23 Thread David Knupp (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5640 to look at the new patch set (#5). Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. ..

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-23 Thread David Knupp (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5640 to look at the new patch set (#4). Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. ..

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-23 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/5640/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-23 Thread David Knupp (Code Review)
David Knupp has uploaded a new patch set (#2). Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. IMPALA-4735: Upgrade pytest in python env to version 2.9.2. The current version of pytest in the

[Impala-ASF-CR] IMPALA-4735: Upgrade pytest in python env to version 2.9.2.

2017-01-23 Thread David Knupp (Code Review)
David Knupp has uploaded a new patch set (#3). Change subject: IMPALA-4735: Upgrade pytest in python env to version 2.9.2. .. IMPALA-4735: Upgrade pytest in python env to version 2.9.2. The current version of pytest in the

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4729: Implement REPLACE() .. Patch Set 1: Hi Zach, please add some regression tests for this. Take a look at exprs.test & test_exprs.py for examples of

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 12: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5585 To

[Impala-ASF-CR] IMPALA-4729: Implement REPLACE()

2017-01-23 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new change for review. http://gerrit.cloudera.org:8080/5776 Change subject: IMPALA-4729: Implement REPLACE() .. IMPALA-4729: Implement REPLACE() This turned out to be slightly non-trivial as

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5739 To unsubscribe,

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5739/6/be/src/runtime/query-state.cc

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5739/6/be/src/runtime/query-state.cc

[Impala-ASF-CR] IMPALA-3909: Populate min/max statistics in Parquet writer

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3909: Populate min/max statistics in Parquet writer .. Patch Set 6: (13 comments) Thanks , I think this will be easier to move forward. I think to make

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-01-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5768/1//COMMIT_MSG

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 12: Code-Review+2 carry -- To view, visit http://gerrit.cloudera.org:8080/5585 To

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 12: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/205/

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, Bharath Vissapragada, Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5585 to look at the new patch set (#12). Change subject: IMPALA-1861: Simplify conditionals with constant conditions

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-01-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5768/1//COMMIT_MSG

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4801: fix heap use after free for MemTracker .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5772 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 13: Code-Review+1 carry +1 -- To view, visit http://gerrit.cloudera.org:8080/5161 To unsubscribe,

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/5161/12/be/src/exprs/agg-fn-evaluator.cc File

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Tim Armstrong (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5161 to look at the new patch set (#13). Change subject: IMPALA-1430: enable codegen for native UDAs .. IMPALA-1430:

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 6: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/204/ -- To

[Impala-ASF-CR] IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges.

2017-01-23 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/5774 Change subject: IMPALA-4263: Fix wrong ommission of agg/analytic hash exchanges. .. IMPALA-4263: Fix wrong ommission of agg/analytic hash

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4801: fix heap use after free for MemTracker .. Patch Set 2: Code-Review+1 Carry +1 -- To view, visit http://gerrit.cloudera.org:8080/5772 To

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4801: fix heap use after free for MemTracker .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Tim Armstrong (Code Review)
Hello Thomas Tauber-Marshall, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5772 to look at the new patch set (#2). Change subject: IMPALA-4801: fix heap use after free for MemTracker ..

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-01-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. Patch Set 9: (1 comment) > I believe the current patchset still has the resource

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-01-23 Thread Thomas Tauber-Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5389 to look at the new patch set (#9). Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5739 To unsubscribe,

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Tim Armstrong (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5739 to look at the new patch set (#5). Change subject: IMPALA-3748: add query-wide resource acquisition step ..

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 12: Code-Review+1 (2 comments)

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 5: Removed the unnecessary call to Prepare() as discussed. -- To view, visit

[Impala-ASF-CR] IMPALA-3909: Populate min/max statistics in Parquet writer

2017-01-23 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3909: Populate min/max statistics in Parquet writer .. Patch Set 6: I removed the support for all but the numerical types from the change. I tried to keep

[Impala-ASF-CR] IMPALA-3909: Populate min/max statistics in Parquet writer

2017-01-23 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#6). Change subject: IMPALA-3909: Populate min/max statistics in Parquet writer .. IMPALA-3909: Populate min/max statistics in Parquet writer Change-Id:

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4801: fix heap use after free for MemTracker .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#4). Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen This change fixes

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#3). Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen This change fixes

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/11/be/src/exprs/agg-fn-evaluator.cc File

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#12). Change subject: IMPALA-1430: enable codegen for native UDAs .. IMPALA-1430: enable codegen for native UDAs This uses the existing infrastructure for codegening builtin UDAs and

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. Patch Set 8: I believe the current patchset still has the resource transfer bug (unless I

[Impala-ASF-CR] IMPALA-3748: add query-wide resource acquisition step

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3748: add query-wide resource acquisition step .. Patch Set 4: Talked offline with Marcel - he pointed out that we shouldn't need to call Prepare() from

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 11: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4801: fix heap use after free for MemTracker .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5772 To

[Impala-ASF-CR] IMPALA-4801: fix heap use after free for MemTracker

2017-01-23 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/5772 Change subject: IMPALA-4801: fix heap use after free for MemTracker .. IMPALA-4801: fix heap use after free for MemTracker This is a

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2017-01-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. Patch Set 8: What's the next step on this one? -- To view, visit

[Impala-ASF-CR] IMPALA-3989: Display skew warning for poorly formatted Parquet files

2017-01-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3989: Display skew warning for poorly formatted Parquet files .. Patch Set 9: One thing I'm a little concerned about, though, is that the user may not be

[Impala-ASF-CR] IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4705, IMPALA-4779, IMPALA-4780: Fix some Expr bugs with codegen .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-3989: Display skew warning for poorly formatted Parquet files

2017-01-23 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3989: Display skew warning for poorly formatted Parquet files .. Patch Set 9: Code-Review+2 (2 comments) If you want to add the config values to the

[Impala-ASF-CR] IMPALA-4792: Fix number of distinct values for a CASE with constant outputs

2017-01-23 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded a new change for review. http://gerrit.cloudera.org:8080/5768 Change subject: IMPALA-4792: Fix number of distinct values for a CASE with constant outputs .. IMPALA-4792: Fix number of distinct

[Impala-ASF-CR] IMPALA-1430: enable codegen for native UDAs

2017-01-23 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-1430: enable codegen for native UDAs .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/5161/11/be/src/exprs/agg-fn-evaluator.cc File

[Impala-ASF-CR] IMPALA-1654: [DOCS] DDL for multiple partitions

2017-01-23 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#5). Change subject: IMPALA-1654: [DOCS] DDL for multiple partitions .. IMPALA-1654: [DOCS] DDL for multiple partitions Syntax and usage notes for ALTER TABLE, COMPUTE STATS, and SHOW

[Impala-ASF-CR] IMPALA-3785: Record query handle for invalid handle

2017-01-23 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3785: Record query handle for invalid handle .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5748/3/be/src/util/debug-util-test.cc File

[Impala-ASF-CR] IMPALA-1861: Simplify conditionals with constant conditions

2017-01-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-1861: Simplify conditionals with constant conditions .. Patch Set 11: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/203/