[Impala-ASF-CR] IMPALA-7200: Fix missing FILESYSTEM PREFIX hitting local dataload

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10803 ) Change subject: IMPALA-7200: Fix missing FILESYSTEM_PREFIX hitting local dataload .. IMPALA-7200: Fix missing

[Impala-ASF-CR] IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion()

2018-06-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10705 ) Change subject: IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion() .. Patch Set 3: (6 comments)

[Impala-ASF-CR] IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion()

2018-06-22 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10705 to look at the new patch set (#4). Change subject: IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion() ..

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion()

2018-06-22 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10705 ) Change subject: IMPALA-6816: minimise calls to GetMinSubscriberTopicVersion() .. Patch Set 3: (6 comments)

[Impala-ASF-CR] IMPALA-7200: Fix missing FILESYSTEM PREFIX hitting local dataload

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10803 ) Change subject: IMPALA-7200: Fix missing FILESYSTEM_PREFIX hitting local dataload .. Patch Set 1: Verified+1 -- To view,

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 10: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. IMPALA-7180: Pin Impala CDH dependencies For

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7201. Support DDL with LocalCatalog enabled

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10806 to review the following change. Change subject: IMPALA-7201. Support DDL with LocalCatalog enabled ..

[Impala-ASF-CR] IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10798 to look at the new patch set (#2). Change subject: IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables

[Impala-ASF-CR] IMPALA-7140 (part 8): support views in LocalCatalog

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10805 to review the following change. Change subject: IMPALA-7140 (part 8): support views in LocalCatalog ..

[Impala-ASF-CR] IMPALA-7149: Disable some tests in the EC build

2018-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10804 Change subject: IMPALA-7149: Disable some tests in the EC build .. IMPALA-7149: Disable some tests in the EC build We temporarily

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 3: Code-Review+2 -- To view, visit

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

2018-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 16: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6642 (Part 2): clean up start-impala-cluster.py

2018-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10780 ) Change subject: IMPALA-6642 (Part 2): clean up start-impala-cluster.py .. IMPALA-6642 (Part 2): clean up start-impala-cluster.py We

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

2018-06-22 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. ( http://gerrit.cloudera.org:8080/6023 ) Change subject: IMPALA-4848: Add WIDTH_BUCKET() function .. Patch Set 16: (3 comments)

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

2018-06-22 Thread anujphadke (Code Review)
Hello Taras Bobrovytsky, Michael Brown, Tim Armstrong, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6023 to look at the new patch set (#16). Change subject: IMPALA-4848: Add WIDTH_BUCKET() function

[Impala-ASF-CR] IMPALA-110: Planner support for multiple distinct aggregations.

2018-06-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has abandoned this change. ( http://gerrit.cloudera.org:8080/10444 ) Change subject: IMPALA-110: Planner support for multiple distinct aggregations. .. Abandoned -- To view, visit

[Impala-ASF-CR] IMPALA-110: Planner support for multiple distinct aggregations.

2018-06-22 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10444 ) Change subject: IMPALA-110: Planner support for multiple distinct aggregations. .. Patch Set 1: This can be abandoned, all of its

[Impala-ASF-CR] IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py

2018-06-22 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/10780 ) Change subject: IMPALA-6642 (Part 2): Add timestamps to start-impala-cluster.py .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-7199: Add scripts to create code coverage reports

2018-06-22 Thread Joe McDonnell (Code Review)
Hello Philip Zeyliger, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10791 to look at the new patch set (#2). Change subject: IMPALA-7199: Add scripts to create code coverage reports ..

[Impala-ASF-CR] Add scripts to create code coverage reports

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10791 ) Change subject: Add scripts to create code coverage reports .. Patch Set 1: (3 comments) Made this into a JIRA IMPALA-7199. There is

[Impala-ASF-CR] IMPALA-7198: [DOCS] Corrected misspelled hints

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10796 ) Change subject: IMPALA-7198: [DOCS] Corrected misspelled hints .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7198: [DOCS] Corrected misspelled hints

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10796 ) Change subject: IMPALA-7198: [DOCS] Corrected misspelled hints .. IMPALA-7198: [DOCS] Corrected misspelled hints

[Impala-ASF-CR] IMPALA-7140 (part 3): load partitions for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10713 ) Change subject: IMPALA-7140 (part 3): load partitions for FS tables .. IMPALA-7140 (part 3): load partitions for FS tables This

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. IMPALA-7140 (part 4): support creating

[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10696 ) Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls .. Patch Set 4: Forgot to respond to the top-level

[Impala-ASF-CR] IMPALA-7198: [DOCS] Corrected misspelled hints

2018-06-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10796 ) Change subject: IMPALA-7198: [DOCS] Corrected misspelled hints .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7198: [DOCS] Corrected misspelled hints

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10796 ) Change subject: IMPALA-7198: [DOCS] Corrected misspelled hints .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-7200: Fix missing FILESYSTEM PREFIX hitting local dataload

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10803 ) Change subject: IMPALA-7200: Fix missing FILESYSTEM_PREFIX hitting local dataload .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-7200: Fix missing FILESYSTEM PREFIX hitting local dataload

2018-06-22 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10803 ) Change subject: IMPALA-7200: Fix missing FILESYSTEM_PREFIX hitting local dataload .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7200: Fix missing FILESYSTEM PREFIX hitting local dataload

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10803 Change subject: IMPALA-7200: Fix missing FILESYSTEM_PREFIX hitting local dataload .. IMPALA-7200: Fix missing FILESYSTEM_PREFIX

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 10: Code-Review+2 -- To view, visit

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

2018-06-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 4: (5 comments) I still need to look at

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

2018-06-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 4: (2 comments) Had a couple of high-level

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 3: Code-Review+2 -- To view, visit

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

2018-06-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10394 ) Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode .. Patch Set 4: I can take over the review once it looks

[Impala-ASF-CR] IMPALA-7191: don't call srand() at random times

2018-06-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10778 ) Change subject: IMPALA-7191: don't call srand() at random times .. Patch Set 2: Seems good to merge -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. Patch Set 4: Verified+1 -- To view,

[Impala-ASF-CR] [DOCS] Update to the workaround for IMPALA-3316

2018-06-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10800 Change subject: [DOCS] Update to the workaround for IMPALA-3316 .. [DOCS] Update to the workaround for IMPALA-3316 Both the date and

[Impala-ASF-CR] IMPALA-7140 (part 3): load partitions for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10713 ) Change subject: IMPALA-7140 (part 3): load partitions for FS tables .. Patch Set 6: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. IMPALA-5900: [DOCS] Doc

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-6802 (part 5): Clean up authorization tests

2018-06-22 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10799 Change subject: IMPALA-6802 (part 5): Clean up authorization tests .. IMPALA-6802 (part 5): Clean up authorization tests The fifth part

[native-toolchain-CR] Patch llvm to fix run-clang-tidy.py output

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10788 ) Change subject: Patch llvm to fix run-clang-tidy.py output .. Patch llvm to fix run-clang-tidy.py output Clang's

[native-toolchain-CR] Patch llvm to fix run-clang-tidy.py output

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10788 ) Change subject: Patch llvm to fix run-clang-tidy.py output .. Patch Set 5: Verified+1 Toolchain package build succeeded on all

[Impala-ASF-CR] IMPALA-3956 - Impala shell should ignore variables in comments

2018-06-22 Thread Adam Holley (Code Review)
Adam Holley has removed Taras Bobrovytsky from this change. ( http://gerrit.cloudera.org:8080/10784 ) Change subject: IMPALA-3956 - Impala shell should ignore variables in comments .. Removed reviewer Taras Bobrovytsky. -- To

[Impala-ASF-CR] IMPALA-3956 - Impala shell should ignore variables in comments

2018-06-22 Thread Adam Holley (Code Review)
Adam Holley has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/10784 ) Change subject: IMPALA-3956 - Impala shell should ignore variables in comments .. IMPALA-3956 - Impala shell should ignore variables in

[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-22 Thread Bharath Vissapragada (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Balazs Jeszenszky, Zoram Thanga, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10696 to look at the new patch set (#4). Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-22 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/10696 ) Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls .. Patch Set 3: (9 comments)

[native-toolchain-CR] Patch llvm to fix run-clang-tidy.py output

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10788 ) Change subject: Patch llvm to fix run-clang-tidy.py output .. Patch Set 5: Code-Review+2 Carry +2 -- To view, visit

[native-toolchain-CR] Patch llvm to fix run-clang-tidy.py output

2018-06-22 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10788 ) Change subject: Patch llvm to fix run-clang-tidy.py output .. Patch Set 4: (1 comment)

[native-toolchain-CR] Patch llvm to fix run-clang-tidy.py output

2018-06-22 Thread Joe McDonnell (Code Review)
Hello Lars Volker, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10788 to look at the new patch set (#5). Change subject: Patch llvm to fix run-clang-tidy.py output ..

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 2: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3956 - Impala shell should ignore variables in comments

2018-06-22 Thread Adam Holley (Code Review)
Adam Holley has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10784 ) Change subject: IMPALA-3956 - Impala shell should ignore variables in comments .. IMPALA-3956 - Impala shell should ignore variables in

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 9: Build started:

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 9: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10798 ) Change subject: IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Alex Rodoni (Code Review)
Hello Juan Yu, Michael Brown, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10795 to look at the new patch set (#2). Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option

[Impala-ASF-CR] IMPALA-4669: [KUTIL] Add kudu util library to the build.

2018-06-22 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/10758 ) Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 1: (5 comments)

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10798 to review the following change. Change subject: IMPALA-7140 (part 7): small fixes to enable most queries on HDFS tables

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10795 ) Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. Patch Set 1: (5 comments)

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-7140 (part 3): load partitions for FS tables

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10713 ) Change subject: IMPALA-7140 (part 3): load partitions for FS tables .. Patch Set 6: Build started:

[Impala-ASF-CR] IMPALA-7140 (part 3): load partitions for FS tables

2018-06-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10713 ) Change subject: IMPALA-7140 (part 3): load partitions for FS tables .. Patch Set 6: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7140 (part 6): fetch column stats for LocalTable

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10797 to review the following change. Change subject: IMPALA-7140 (part 6): fetch column stats for LocalTable

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10749 to look at the new patch set (#3). Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Hello Tianyi Wang, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10735 to look at the new patch set (#4). Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 2: (5 comments)

[Impala-ASF-CR] IMPALA-7198: [DOCS] Corrected misspelled hints

2018-06-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10796 Change subject: IMPALA-7198: [DOCS] Corrected misspelled hints .. IMPALA-7198: [DOCS] Corrected misspelled hints Change-Id:

[Impala-ASF-CR] IMPALA-7140 (part 4): support creating descriptors for FS tables

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10735 ) Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables .. Patch Set 3: (6 comments)

[Impala-ASF-CR] IMPALA-7140 (part 5): support fetching file info for FS tables

2018-06-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10749 ) Change subject: IMPALA-7140 (part 5): support fetching file info for FS tables .. Patch Set 2: Code-Review+2 (6 comments)

[Impala-ASF-CR] IMPALA-5900: [DOCS] Doc fe service threads startup option

2018-06-22 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10795 Change subject: IMPALA-5900: [DOCS] Doc fe_service_threads startup option .. IMPALA-5900: [DOCS] Doc fe_service_threads startup option

[Impala-ASF-CR] Add scripts to create code coverage reports

2018-06-22 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10791 ) Change subject: Add scripts to create code coverage reports .. Patch Set 1: Code-Review+1 (3 comments) Looks fine to me. I'd love

[Impala-ASF-CR] IMPALA-7121: Clean up partitionIds from HdfsTable

2018-06-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 6: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Fredy Wijaya (Code Review)
Fredy Wijaya has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. IMPALA-7180: Pin Impala CDH dependencies For IMPALA_MINICLUSTER_PROFILE=3

[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-22 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10696 ) Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10790 ) Change subject: IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0 .. IMPALA-5763: [DOCS] Warn against setting

[Impala-ASF-CR] IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10790 ) Change subject: IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0 .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-22 Thread Vuk Ercegovac (Code Review)
Vuk Ercegovac has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 18: (2 comments)

[Impala-ASF-CR] IMPALA-7180: Pin Impala CDH dependencies

2018-06-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/10748 ) Change subject: IMPALA-7180: Pin Impala CDH dependencies .. Patch Set 7: Code-Review+2 (1 comment) Had one nit, otherwise LGTM.

[Impala-ASF-CR] IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10790 ) Change subject: IMPALA-5763: [DOCS] Warn against setting --logbufsecs to 0 .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-22 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 18: (9 comments)

[Impala-ASF-CR] [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls

2018-06-22 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10696 ) Change subject: [DRAFT] IMPALA-6189: Add thread watchdogs for HDFS IO calls .. Patch Set 3: (7 comments) Hi! Thanks for doing

[Impala-ASF-CR] IMPALA-7175: deflake check for failed impalad

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10745 ) Change subject: IMPALA-7175: deflake check for failed impalad .. IMPALA-7175: deflake check for failed impalad

[Impala-ASF-CR] IMPALA-7175: deflake check for failed impalad

2018-06-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10745 ) Change subject: IMPALA-7175: deflake check for failed impalad .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-22 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/10543 ) Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location .. Patch Set 18: (10 comments)

[Impala-ASF-CR] IMPALA-6119: Fix issue with multiple partitions sharing same location

2018-06-22 Thread Gabor Kaszab (Code Review)
Hello Bharath Vissapragada, Michael Brown, Zoltan Borok-Nagy, Sailesh Mukil, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10543 to look at the new patch set (#18). Change subject: IMPALA-6119: Fix issue with multiple partitions sharing

  1   2   >