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

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build. .. Patch Set 21: PS21 passed all tests. clang-tidy issue is still outstanding, as there is no obvious

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded a new patch set (#3). Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. IMPALA-5520: TopN node periodically reclaims old allocations Currently TopN retains old string

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (2 comments)

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

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build. .. Patch Set 22: PS22 takes the blunt-instrument approach to preventing clang-tidy from running on

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. Patch Set 1: (19 comments) http://gerrit.cloudera.org:8080/#/c/7400/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. IMPALA-5627: fix dropped statuses in HDFS writers The change is mostly mechanical - added Status returns

[Impala-ASF-CR] IMPALA-5116: Removal of uses for the deprecated functions in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has abandoned this change. Change subject: IMPALA-5116: Removal of uses for the deprecated functions in gutil .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7413 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 5: Can you add this information to the root (/) web page for each daemon? I think it's very

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (9 comments) Few more in addition to MJ's comments.

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

2017-07-13 Thread Henry Robinson (Code Review)
Hello Impala Public Jenkins, Michael Ho, Matthew Jacobs, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5715 to look at the new patch set (#22). Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build.

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/7332/3/be/src/catalog/catalog.cc File

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/7372/4/be/src/exec/hdfs-avro-table-writer.h

[Impala-ASF-CR] test

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has uploaded a new change for review. http://gerrit.cloudera.org:8080/7412 Change subject: test .. test Change-Id: I8e3f1e28004e9d2a7526c83e00fcd8ecd2e0abc2 --- M README.md 1 file changed, 2 insertions(+), 0

[Impala-ASF-CR] IMPALA-5116: Removal of uses for the deprecated functions in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has uploaded a new change for review. http://gerrit.cloudera.org:8080/7413 Change subject: IMPALA-5116: Removal of uses for the deprecated functions in gutil .. IMPALA-5116: Removal of uses for the deprecated

[Impala-ASF-CR] IMPALA-5407: Fix crash in HdfsSequenceTableWriter

2017-07-13 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#2). Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter .. IMPALA-5407: Fix crash in HdfsSequenceTableWriter The following use of sequence file writer can lead to a

[Impala-ASF-CR] IMPALA-5407: Fix crash in HdfsSequenceTableWriter

2017-07-13 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#2). Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter .. IMPALA-5407: Fix crash in HdfsSequenceTableWriter The following use of sequence file writer can lead to a

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/7332/3/be/src/catalog/catalog.cc File

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-13 Thread Gabor Kaszab (Code Review)
Gabor Kaszab has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 5: comments incorporated -- To view, visit http://gerrit.cloudera.org:8080/7363 To

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. Patch Set 3: (7 comments)

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#4). Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. IMPALA-5650: Make sum_init_zero a SUM function The recent Parquet count(*) optimization (IMPALA-5036)

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 30: Code-Review+2 (4 comments) +2 assuming we do the addition flag in a separate change. Good

[Impala-ASF-CR] IMPALA-4674: Part 3: fix null-aware anti join

2017-07-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4674: Part 3: fix null-aware anti join .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/7367/1//COMMIT_MSG Commit Message: PS1, Line 13:

[native-toolchain-CR] IMPALA-5659: Build shared libs for gflags

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7415 Change subject: IMPALA-5659: Build shared libs for gflags .. IMPALA-5659: Build shared libs for gflags Change-Id:

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/7404/3/be/src/exec/partitioned-aggregation-node.cc

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7416/1//COMMIT_MSG

[native-toolchain-CR] IMPALA-5659: Build shared libs for gflags

2017-07-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5659: Build shared libs for gflags .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7415 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. Patch Set 4: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/859/ -- To view,

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7404 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 26: (1 comment) http://gerrit.cloudera.org:8080/#/c/5801/26/be/src/runtime/query-state.cc File

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has uploaded a new patch set (#2). Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. IMPALA-5116: Remove deprecated hash_* types in gutil The following class templates are substituted from

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has posted comments on this change. Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7414/1/be/src/gutil/hash/hash.h File

[Impala-ASF-CR] IMPALA-4703: reservation denial debug action

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4703: reservation denial debug action .. Patch Set 7: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7022 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4674: Part 3: fix null-aware anti join

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 3: fix null-aware anti join .. Patch Set 1: (7 comments) http://gerrit.cloudera.org:8080/#/c/7367/1//COMMIT_MSG Commit Message: PS1, Line

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#2). Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS Bugs:

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-5659: Begin standardizing treatment of thirdparty libraries

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7418 Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty libraries .. IMPALA-5659: Begin standardizing treatment of

[Impala-ASF-CR] IMPALA-5158,IMPALA-5236: account for unused buffer pool reservations

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#3). Change subject: IMPALA-5158,IMPALA-5236: account for unused buffer pool reservations .. IMPALA-5158,IMPALA-5236: account for unused buffer pool reservations We were missing

[Impala-ASF-CR] IMPALA-4674: Part 3: fix null-aware anti join

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-4674: Part 3: fix null-aware anti join .. IMPALA-4674: Part 3: fix null-aware anti join Part 2 regressed NAAJ by tightening up the spilling invariants

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-07-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7414/1/be/src/gutil/hash/hash.h File

[Impala-ASF-CR] IMPALA-4674: Part 1: remove old aggs and joins

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 1: remove old aggs and joins .. Patch Set 9: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7102 To unsubscribe,

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5801 to look at the new patch set (#32). Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool ..

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 32: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/5801 To

[Impala-ASF-CR] IMPALA-5116: Removal of uses for the deprecated functions in gutil

2017-07-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-5116: Removal of uses for the deprecated functions in gutil .. Patch Set 1: (7 comments) THank you for sending this patch! A few questions:

[Impala-ASF-CR] [DOCS] Fold some lines

2017-07-13 Thread John Russell (Code Review)
John Russell has uploaded a new change for review. http://gerrit.cloudera.org:8080/7417 Change subject: [DOCS] Fold some lines .. [DOCS] Fold some lines Putting on a separate line causes an extra leading blank line in PDF

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5801 to look at the new patch set (#31). Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool ..

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 31: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5801 To

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded a new change for review. http://gerrit.cloudera.org:8080/7419 Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. IMPALA-4276: Profile displays non-default

[Impala-ASF-CR] IMPALA-5511: Add process start time to debug web page

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5511: Add process start time to debug web page .. Patch Set 5: To that end, it might be better to define a single process-start-time metric, and to

[Impala-ASF-CR] IMPALA-5521: TopN node periodically reclaims old allocations

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded a new patch set (#2). Change subject: IMPALA-5521: TopN node periodically reclaims old allocations .. IMPALA-5521: TopN node periodically reclaims old allocations Currently TopN retains old string

[native-toolchain-CR] IMPALA-5659: Build shared libs for gflags

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: IMPALA-5659: Build shared libs for gflags .. IMPALA-5659: Build shared libs for gflags Change-Id: If1842768f21444ccf231cfd3fe5e51a4ca0d7244 --- M

[native-toolchain-CR] IMPALA-5659: Build shared libs for gflags

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5659: Build shared libs for gflags .. Patch Set 1: Verified+1 Passed a toolchain build -- To view, visit http://gerrit.cloudera.org:8080/7415 To

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded a new patch set (#4). Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. IMPALA-5520: TopN node periodically reclaims old allocations Currently TopN retains old string

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/7400/3/be/src/exec/topn-node.cc

[Impala-ASF-CR] IMPALA-5520: TopN node periodically reclaims old allocations

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5520: TopN node periodically reclaims old allocations .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7400/3/be/src/exec/topn-node.h

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7419/1//COMMIT_MSG

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7404 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5650: Make sum init zero a SUM function

2017-07-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5650: Make sum_init_zero a SUM function .. IMPALA-5650: Make sum_init_zero a SUM function The recent Parquet count(*) optimization

[Impala-ASF-CR] IMPALA-5394: Handle blocked HS2 connections

2017-07-13 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-5394: Handle blocked HS2 connections .. Patch Set 3: (1 comment) Patch looks pretty good. It would be great to still respect fe_service_threads, if

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has posted comments on this change. Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/7414/3/be/src/gutil/hash/hash.h File

[Impala-ASF-CR] IMPALA-5116: Remove deprecated hash * types in gutil

2017-07-13 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has uploaded a new patch set (#3). Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil .. IMPALA-5116: Remove deprecated hash_* types in gutil The following class templates are substituted from

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#30). Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. IMPALA-4674: Part 2: port backend exec to BufferPool Always create global BufferPool at startup using 80%

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-13 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 29: (24 comments)

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-13 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/7393/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread anujphadke (Code Review)
anujphadke has uploaded a new patch set (#3). Change subject: IMPALA-5582: Store sentry privileges in lower case .. IMPALA-5582: Store sentry privileges in lower case Privileges granted to a db/table whose name contains upper

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-13 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 3: Discussed this issue with MJ. This issue does occur due to inconsistent handling of casing.

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-13 Thread anujphadke (Code Review)
anujphadke has uploaded a new patch set (#3). Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. IMPALA-5586: Null-aware anti-join can take a long time to cancel Queries with a null-aware