[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 6: Build started:

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Sailesh Mukil (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10843 to look at the new patch set (#6). Change subject: IMPALA-4784: Remove InProcessStatestore

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 6: Code-Review+2 Hit a clang-tidy issue. Rebase, carry +2. --

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 13: Verified-1 Build

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10744 ) Change subject: IMPALA-1760: Implement shutdown command .. Patch Set 7: (19 comments)

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10823 ) Change subject: IMPALA-7190: Remove unsupported format writer support .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. IMPALA-6352: Dump backtrace on failure of

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Bikramjeet Vig (Code Review)
Hello Lars Volker, Tim Armstrong, Csaba Ringhofer, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10823 to look at the new patch set (#3). Change subject: IMPALA-7190: Remove unsupported format writer support

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10823 ) Change subject: IMPALA-7190: Remove unsupported format writer support .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10744 ) Change subject: IMPALA-1760: Implement shutdown command .. Patch Set 8: (13 comments) still need to review tests

[Impala-ASF-CR] IMPALA-6034: Add Cpu and scanned bytes limits per query

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10415 ) Change subject: IMPALA-6034: Add Cpu and scanned bytes limits per query .. Patch Set 6: I'm going to take over this patch from

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 5: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 13: Code-Review+2 -- To

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 13: Build started:

[Impala-ASF-CR] IMPALA-5486: Port ReportExecStatus() RPC to use KRPC

2018-07-02 Thread Michael Ho (Code Review)
Michael Ho has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10855 Change subject: IMPALA-5486: Port ReportExecStatus() RPC to use KRPC .. IMPALA-5486: Port ReportExecStatus() RPC to use KRPC This change

[Impala-ASF-CR] IMPALA-7237: handle hex digits in ParseSmaps()

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10853 Change subject: IMPALA-7237: handle hex digits in ParseSmaps() .. IMPALA-7237: handle hex digits in ParseSmaps() Testing: Manual.

[Impala-ASF-CR] [DOCS] Clarification on admission control and DDL statements

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10829 ) Change subject: [DOCS] Clarification on admission control and DDL statements .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5981: [DOCS] Documented SET=""

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10816 ) Change subject: IMPALA-5981: [DOCS] Documented SET="" .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 12: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10823 ) Change subject: IMPALA-7190: Remove unsupported format writer support .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 12: Code-Review+1 -- To view,

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] [DOCS] Corrected the supported values for parquet array resolution

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10840 ) Change subject: [DOCS] Corrected the supported values for parquet_array_resolution .. [DOCS] Corrected the supported

[Impala-ASF-CR] [DOCS] Corrected the supported values for parquet array resolution

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10840 ) Change subject: [DOCS] Corrected the supported values for parquet_array_resolution .. Patch Set 1: Verified+1 -- To view,

[Impala-ASF-CR] [DOCS] Corrected the supported values for parquet array resolution

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10840 ) Change subject: [DOCS] Corrected the supported values for parquet_array_resolution .. Patch Set 1: Build started:

[Impala-ASF-CR] [DOCS] Corrected the supported values for parquet array resolution

2018-07-02 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10840 ) Change subject: [DOCS] Corrected the supported values for parquet_array_resolution .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has uploaded a new patch set (#12). ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. IMPALA-6625: Skip computing parquet

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 12: (2 comments)

[Impala-ASF-CR] IMPALA-6883: [DOCS] Refactor impala authorization doc

2018-07-02 Thread Alex Rodoni (Code Review)
Hello Fredy Wijaya, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10786 to look at the new patch set (#4). Change subject: IMPALA-6883: [DOCS] Refactor impala_authorization doc

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Bikramjeet Vig (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10851 to look at the new patch set (#2). Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample ..

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 1: (3 comments)

[Impala-ASF-CR] IMPALA-7185: low statestore custom cluster interval

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10845 ) Change subject: IMPALA-7185: low statestore custom cluster interval .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7185: low statestore custom cluster interval

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10845 ) Change subject: IMPALA-7185: low statestore custom cluster interval .. IMPALA-7185: low statestore custom cluster

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 5: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10851 ) Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. Patch Set 1: (3 comments) The approach seems good, just

[Impala-ASF-CR] IMPALA-6352: Dump backtrace on failure of TestTableSample

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10851 Change subject: IMPALA-6352: Dump backtrace on failure of TestTableSample .. IMPALA-6352: Dump backtrace on failure of TestTableSample

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. Patch Set 11: Build started:

[Impala-ASF-CR] IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans

2018-07-02 Thread Pooja Nilangekar (Code Review)
Pooja Nilangekar has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/10704 ) Change subject: IMPALA-6625: Skip computing parquet conjuncts for non-Parquet scans .. IMPALA-6625: Skip computing parquet

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 3: (3 comments)

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Sailesh Mukil (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10843 to look at the new patch set (#4). Change subject: IMPALA-4784: Remove InProcessStatestore .. IMPALA-4784: Remove

[Impala-ASF-CR] IMPALA-6086: Use of permanent function should require SELECT privilege on DB

2018-07-02 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10850 ) Change subject: IMPALA-6086: Use of permanent function should require SELECT privilege on DB .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 12: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. IMPALA-110 (part 2): Refactor

[Impala-ASF-CR] IMPALA-6086: Use of permanent function should require SELECT privilege on DB

2018-07-02 Thread Zoram Thanga (Code Review)
Zoram Thanga has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10850 Change subject: IMPALA-6086: Use of permanent function should require SELECT privilege on DB .. IMPALA-6086: Use of permanent function

[Impala-ASF-CR] IMPALA-6086: Use of permanent function should require SELECT privilege on DB

2018-07-02 Thread Zoram Thanga (Code Review)
Zoram Thanga has abandoned this change. ( http://gerrit.cloudera.org:8080/10842 ) Change subject: IMPALA-6086: Use of permanent function should require SELECT privilege on DB .. Abandoned Raising a new review. -- To view,

[Impala-ASF-CR] IMPALA-6086: Use of permanent function should require SELECT privilege on DB

2018-07-02 Thread Zoram Thanga (Code Review)
Zoram Thanga has posted comments on this change. ( http://gerrit.cloudera.org:8080/10842 ) Change subject: IMPALA-6086: Use of permanent function should require SELECT privilege on DB .. Patch Set 1: (3 comments) Abandoning

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10823 ) Change subject: IMPALA-7190: Remove unsupported format writer support .. Patch Set 2: Code-Review+1 Carrying over +1. Minor changes,

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Bikramjeet Vig (Code Review)
Hello Lars Volker, Tim Armstrong, Csaba Ringhofer, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10823 to look at the new patch set (#2). Change subject: IMPALA-7190: Remove unsupported format writer support

[Impala-ASF-CR] IMPALA-7190: Remove unsupported format writer support

2018-07-02 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/10823 ) Change subject: IMPALA-7190: Remove unsupported format writer support .. Patch Set 1: (4 comments)

[Impala-ASF-CR] IMPALA-4784: Remove InProcessStatestore

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10843 ) Change subject: IMPALA-4784: Remove InProcessStatestore .. Patch Set 3: (3 comments) Seems good, thanks for the cleanup!

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10744 ) Change subject: IMPALA-1760: Implement shutdown command .. Patch Set 7: (6 comments) Fixed a few minor things in the test code that

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Tim Armstrong (Code Review)
Hello Fredy Wijaya, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10744 to look at the new patch set (#8). Change subject: IMPALA-1760: Implement shutdown command ..

[Impala-ASF-CR] IMPALA-7185: low statestore custom cluster interval

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10845 ) Change subject: IMPALA-7185: low statestore custom cluster interval .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-7185: low statestore custom cluster interval

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10845 ) Change subject: IMPALA-7185: low statestore custom cluster interval .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10744 ) Change subject: IMPALA-1760: Implement shutdown command .. Patch Set 7: (5 comments)

[Impala-ASF-CR] IMPALA-7185: low statestore custom cluster interval

2018-07-02 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10845 ) Change subject: IMPALA-7185: low statestore custom cluster interval .. Patch Set 3: Code-Review+2 This looks good to me. -- To

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-07-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 12: Build started:

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-07-02 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 12: Code-Review+2 carrying forward -- To

[Impala-ASF-CR] IMPALA-110 (part 2): Refactor PartitionedAggregationNode

2018-07-02 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 11: Code-Review+2 carrying forward -- To

[Impala-ASF-CR] IMPALA-6883: [DOCS] Refactor impala authorization doc

2018-07-02 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10786 ) Change subject: IMPALA-6883: [DOCS] Refactor impala_authorization doc .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-6271: Impala daemon should log a message when it's being shut down

2018-07-02 Thread Pranay Singh (Code Review)
Pranay Singh has posted comments on this change. ( http://gerrit.cloudera.org:8080/10847 ) Change subject: IMPALA-6271: Impala daemon should log a message when it's being shut down .. Patch Set 1: > How does your change

[Impala-ASF-CR] IMPALA-6271: Impala daemon should log a message when it's being shut down

2018-07-02 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10847 ) Change subject: IMPALA-6271: Impala daemon should log a message when it's being shut down .. Patch Set 1: How does your change

[Impala-ASF-CR] IMPALA-7231: group plan nodes into pipelines

2018-07-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10848 ) Change subject: IMPALA-7231: group plan nodes into pipelines .. IMPALA-7231: group plan nodes into pipelines This adds some

[Impala-ASF-CR] IMPALA-1760: Implement shutdown command

2018-07-02 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10744 to look at the new patch set (#7). Change subject: IMPALA-1760: Implement shutdown command .. IMPALA-1760: Implement

[Impala-ASF-CR] IMPALA-6918: Implement COMMENT ON COLUMN

2018-07-02 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10754 ) Change subject: IMPALA-6918: Implement COMMENT ON COLUMN .. Patch Set 6: Code-Review+1 Updated the commit message. Carry +1. -- To

[Impala-ASF-CR] IMPALA-6918: Implement COMMENT ON COLUMN

2018-07-02 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/10754 ) Change subject: IMPALA-6918: Implement COMMENT ON COLUMN .. IMPALA-6918: Implement COMMENT ON COLUMN This patch implements updating

[Impala-ASF-CR] IMPALA-6918: Implement COMMENT ON COLUMN

2018-07-02 Thread Adam Holley (Code Review)
Adam Holley has posted comments on this change. ( http://gerrit.cloudera.org:8080/10754 ) Change subject: IMPALA-6918: Implement COMMENT ON COLUMN .. Patch Set 5: Code-Review+1 Only nit is in the commit message, the database