[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7056: [DOCS] ALTER TABLE only affects the future inserts

2018-06-15 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10743 ) Change subject: IMPALA-7056: [DOCS] ALTER TABLE only affects the future inserts .. Patch Set 1: Hi Tim, Could you assign a dev for this

[Impala-ASF-CR] IMPALA-7056: [DOCS] ALTER TABLE only affects the future inserts

2018-06-15 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10743 Change subject: IMPALA-7056: [DOCS] ALTER TABLE only affects the future inserts .. IMPALA-7056: [DOCS] ALTER TABLE only affects the

[Impala-ASF-CR] IMPALA-7169: Prevent HDFS from checkpointing trash until 3000 AD

2018-06-15 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10742 Change subject: IMPALA-7169: Prevent HDFS from checkpointing trash until 3000 AD .. IMPALA-7169: Prevent HDFS from checkpointing trash

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

2018-06-15 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: Following diff prevents it from crashing

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

2018-06-15 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: (2 comments) I ran a local stress test with

[Impala-ASF-CR] IMPALA-7171: [DOCS] Hints for Kudu insert and upsert

2018-06-15 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10737 Change subject: IMPALA-7171: [DOCS] Hints for Kudu insert and upsert .. IMPALA-7171: [DOCS] Hints for Kudu insert and upsert Change-Id:

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10690 ) Change subject: IMPALA-7046: introduce "global" debug_actions .. Patch Set 6: (1 comment) This is a rebase but required fix up to remove

[Impala-ASF-CR] IMPALA-7046: introduce "global" debug actions

2018-06-15 Thread Dan Hecht (Code Review)
Hello Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10690 to look at the new patch set (#6). Change subject: IMPALA-7046: introduce "global" debug_actions .. IMPALA-7046:

[Impala-ASF-CR] IMPALA-110 (part 1): Refactor ExecNode::buffer pool client

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10493 ) Change subject: IMPALA-110 (part 1): Refactor ExecNode::buffer_pool_client_ .. Patch Set 2: Code-Review+1 -- To view, visit

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

2018-06-15 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 2: > (14 comments) > > While developing this

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

2018-06-15 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: (2 comments)

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: ok -- To view, visit

[Impala-ASF-CR] IMPALA-110 (part 1): Refactor ExecNode::buffer pool client

2018-06-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10493 ) Change subject: IMPALA-110 (part 1): Refactor ExecNode::buffer_pool_client_ .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: I couldn't think of anything

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: Code-Review+2 Seems better. Is

[Impala-ASF-CR] IMPALA-110 (part 1): Refactor ExecNode::buffer pool client

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10493 ) Change subject: IMPALA-110 (part 1): Refactor ExecNode::buffer_pool_client_ .. Patch Set 2: Code-Review+2 Thanks. I think this makes

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: Anuj pointed out that I'd forgot

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. Patch Set 2: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/10736 ) Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. IMPALA-7179:

[Impala-ASF-CR] IMPALA-2746: part 1: enable LSAN for many backend tests

2018-06-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10668 ) Change subject: IMPALA-2746: part 1: enable LSAN for many backend tests .. Patch Set 9: Code-Review+2 (1 comment) This all looks

[Impala-ASF-CR] IMPALA-2746: part 1: enable LSAN for many backend tests

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10668 ) Change subject: IMPALA-2746: part 1: enable LSAN for many backend tests .. Patch Set 8: (2 comments)

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

2018-06-15 Thread Thomas Marshall (Code Review)
Hello Tim Armstrong, Alex Behm, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10394 to look at the new patch set (#2). Change subject: IMPALA-110 (part 2): Refactor PartitionedAggregationNode

[Impala-ASF-CR] IMPALA-110 (part 1): Refactor ExecNode::buffer pool client

2018-06-15 Thread Thomas Marshall (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10493 to look at the new patch set (#2). Change subject: IMPALA-110 (part 1): Refactor ExecNode::buffer_pool_client_ ..

[Impala-ASF-CR] IMPALA-7161: Fix impala-config.sh's handling of JAVA HOME

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10702 ) Change subject: IMPALA-7161: Fix impala-config.sh's handling of JAVA_HOME .. Patch Set 3: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-7179: allow multiple scratch dirs per device=true by default

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10736 Change subject: IMPALA-7179: allow_multiple_scratch_dirs_per_device=true by default .. IMPALA-7179:

[Impala-ASF-CR] IMPALA-7137. Support configuring Frontend to use LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10629 ) Change subject: IMPALA-7137. Support configuring Frontend to use LocalCatalog .. Patch Set 7: Verified+1 -- To view, visit

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

2018-06-15 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/10735 to review the following change. Change subject: IMPALA-7140 (part 4): support creating descriptors for FS tables

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

2018-06-15 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/10713 to look at the new patch set (#3). Change subject: IMPALA-7140 (part 3): load partitions for FS tables

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

2018-06-15 Thread Todd Lipcon (Code Review)
Todd Lipcon 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 2: (4 comments)

[Impala-ASF-CR] IMPALA-6812: Fix flaky Kudu scan tests

2018-06-15 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10503 ) Change subject: IMPALA-6812: Fix flaky Kudu scan tests .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-6812: Fix flaky Kudu scan tests

2018-06-15 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/10503 ) Change subject: IMPALA-6812: Fix flaky Kudu scan tests .. Patch Set 4: Code-Review+2 (1 comment)

[Impala-ASF-CR] Optimize dependencies for Codegen

2018-06-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10688 ) Change subject: Optimize dependencies for Codegen .. Patch Set 1: > Looks like the current way was done as a result of IMPALA-1896.

[Impala-ASF-CR] Optimize dependencies for Codegen

2018-06-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has abandoned this change. ( http://gerrit.cloudera.org:8080/10688 ) Change subject: Optimize dependencies for Codegen .. Abandoned This has a flawed dependency graph. -- To view, visit

[Impala-ASF-CR] [DOCS] Fixed the DITA concept ID for the new impala disable codegen rows threshold

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10733 ) Change subject: [DOCS] Fixed the DITA concept ID for the new impala_disable_codegen_rows_threshold .. Patch Set 1: Verified+1

[Impala-ASF-CR] [DOCS] Fixed the DITA concept ID for the new impala disable codegen rows threshold

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10733 ) Change subject: [DOCS] Fixed the DITA concept ID for the new impala_disable_codegen_rows_threshold .. [DOCS] Fixed

[Impala-ASF-CR] IMPALA-2746: part 1: enable LSAN for many backend tests

2018-06-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10668 ) Change subject: IMPALA-2746: part 1: enable LSAN for many backend tests .. Patch Set 8: (2 comments) I'm basically ready to +2 this.

[Impala-ASF-CR] [DOCS] Fixed the DITA concept ID for the new impala disable codegen rows threshold

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10733 ) Change subject: [DOCS] Fixed the DITA concept ID for the new impala_disable_codegen_rows_threshold .. Patch Set 1: Build

[Impala-ASF-CR] [DOCS] Fixed the DITA concept ID for the new impala disable codegen rows threshold

2018-06-15 Thread Alex Rodoni (Code Review)
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10733 ) Change subject: [DOCS] Fixed the DITA concept ID for the new impala_disable_codegen_rows_threshold .. Patch Set 1: Code-Review+2 --

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

2018-06-15 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 2: (4 comments)

[Impala-ASF-CR] [DOCS] Fixed the DITA concept ID for the new impala disable codegen rows threshold

2018-06-15 Thread Alex Rodoni (Code Review)
Alex Rodoni has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10733 Change subject: [DOCS] Fixed the DITA concept ID for the new impala_disable_codegen_rows_threshold .. [DOCS] Fixed the DITA concept ID

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

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10442 ) Change subject: IMPALA-6802 (part 4): Clean up authorization tests .. Patch Set 5: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7161: Fix impala-config.sh's handling of JAVA HOME

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10702 ) Change subject: IMPALA-7161: Fix impala-config.sh's handling of JAVA_HOME .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-7137. Support configuring Frontend to use LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10629 ) Change subject: IMPALA-7137. Support configuring Frontend to use LocalCatalog .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-7140 (part 2). Create skeleton for LocalFsTable

2018-06-15 Thread Todd Lipcon (Code Review)
Hello Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10712 to look at the new patch set (#3). Change subject: IMPALA-7140 (part 2). Create skeleton for LocalFsTable ..

[Impala-ASF-CR] IMPALA-7137. Support configuring Frontend to use LocalCatalog

2018-06-15 Thread Todd Lipcon (Code Review)
Hello Impala Public Jenkins, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10629 to look at the new patch set (#7). Change subject: IMPALA-7137. Support configuring Frontend to use LocalCatalog

[Impala-ASF-CR] IMPALA-7141 (part 1): clean up handling of default/dummy partition

2018-06-15 Thread Todd Lipcon (Code Review)
Hello Gabor Kaszab, Zoltan Borok-Nagy, Vuk Ercegovac, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10711 to look at the new patch set (#3). Change subject: IMPALA-7141 (part 1): clean up handling of default/dummy partition

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

2018-06-15 Thread Todd Lipcon (Code Review)
Todd Lipcon 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 2: (16 comments)

[Impala-ASF-CR] IMPALA-7161: Fix impala-config.sh's handling of JAVA HOME

2018-06-15 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10702 ) Change subject: IMPALA-7161: Fix impala-config.sh's handling of JAVA_HOME .. Patch Set 3: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7161: Fix impala-config.sh's handling of JAVA HOME

2018-06-15 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/10702 ) Change subject: IMPALA-7161: Fix impala-config.sh's handling of JAVA_HOME .. Patch Set 2: (4 comments)

[Impala-ASF-CR] IMPALA-7161: Fix impala-config.sh's handling of JAVA HOME

2018-06-15 Thread Joe McDonnell (Code Review)
Hello Tianyi Wang, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10702 to look at the new patch set (#3). Change subject: IMPALA-7161: Fix impala-config.sh's handling of JAVA_HOME

[Impala-ASF-CR] IMPALA-7140 (part 2). Create skeleton for LocalFsTable

2018-06-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10712 ) Change subject: IMPALA-7140 (part 2). Create skeleton for LocalFsTable .. Patch Set 2: (3 comments)

[Impala-ASF-CR] IMPALA-7141 (part 1): clean up handling of default/dummy partition

2018-06-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10711 ) Change subject: IMPALA-7141 (part 1): clean up handling of default/dummy partition .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-6812: Fix flaky Kudu scan tests

2018-06-15 Thread Thomas Marshall (Code Review)
Thomas Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/10503 ) Change subject: IMPALA-6812: Fix flaky Kudu scan tests .. Patch Set 4: Code-Review+1 (1 comment) carrying forward

[Impala-ASF-CR] IMPALA-6812: Fix flaky Kudu scan tests

2018-06-15 Thread Thomas Marshall (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10503 to look at the new patch set (#4). Change subject: IMPALA-6812: Fix flaky Kudu scan tests

[Impala-ASF-CR] IMPALA-2746: part 1: enable LSAN for many backend tests

2018-06-15 Thread Tim Armstrong (Code Review)
Hello Philip Zeyliger, Joe McDonnell, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10668 to look at the new patch set (#8). Change subject: IMPALA-2746: part 1: enable LSAN for many backend tests

[Impala-ASF-CR] IMPALA-7140 (part 1). Support fetching schema info in LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10630 ) Change subject: IMPALA-7140 (part 1). Support fetching schema info in LocalCatalog .. Patch Set 7: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-7140 (part 1). Support fetching schema info in LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10630 ) Change subject: IMPALA-7140 (part 1). Support fetching schema info in LocalCatalog .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-7130: impala-shell -b / --kerberos host fqdn flag overrides value passed in via -i / --impalad

2018-06-15 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/10580 ) Change subject: IMPALA-7130: impala-shell -b / --kerberos_host_fqdn flag overrides value passed in via -i / --impalad .. Patch Set

[Impala-ASF-CR] IMPALA-6969: add AC last queued cause to profile

2018-06-15 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10731 Change subject: IMPALA-6969: add AC last queued cause to profile .. IMPALA-6969: add AC last queued cause to profile The reason is

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

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10442 ) Change subject: IMPALA-6802 (part 4): Clean up authorization tests .. Patch Set 5: Build started:

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

2018-06-15 Thread Adam Holley (Code Review)
Adam Holley has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/10442 ) Change subject: IMPALA-6802 (part 4): Clean up authorization tests .. IMPALA-6802 (part 4): Clean up authorization tests The fourth part

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

2018-06-15 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/10654 ) Change subject: IMPALA-7121: Clean up partitionIds_ from HdfsTable .. Patch Set 3: -Code-Review Todd, I see you have a +2 on

[Impala-ASF-CR] IMPALA-7141 (part 1): clean up handling of default/dummy partition

2018-06-15 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. ( http://gerrit.cloudera.org:8080/10711 ) Change subject: IMPALA-7141 (part 1): clean up handling of default/dummy partition .. Patch Set 2: Code-Review+1 (2 comments)

[Impala-ASF-CR] Fail cleanly when in process server can't bind

2018-06-15 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/10726 ) Change subject: Fail cleanly when in process server can't bind .. Patch Set 1: Code-Review+1 I thought a bit about not too

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

2018-06-15 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 12: Patch set 12 is adjusting

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

2018-06-15 Thread Gabor Kaszab (Code Review)
Hello Bharath Vissapragada, 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 (#12). Change subject: IMPALA-6119: Fix issue with multiple partitions sharing same location

[Impala-ASF-CR] IMPALA-7137. Support configuring Frontend to use LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10629 ) Change subject: IMPALA-7137. Support configuring Frontend to use LocalCatalog .. Patch Set 6: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7137. Support configuring Frontend to use LocalCatalog

2018-06-15 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/10629 ) Change subject: IMPALA-7137. Support configuring Frontend to use LocalCatalog .. Patch Set 6: Build started: