anujphadke has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 5:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/7332/4//COMMIT_MSG
Commit Message:
PS4, Line 9
Hello Matthew Jacobs,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7332
to look at the new patch set (#5).
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
IMPALA-5
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Build failed because not all the Jenkins executors had cleaned their toolcha
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-ver
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5520: TopN node periodically reclaims old allocations
..
IMPALA-5520: TopN node periodically reclaims old allocations
Currently TopN reta
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5520: TopN node periodically reclaims old allocations
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7400
To unsubs
Zach Amsden has posted comments on this change.
Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
..
Patch Set 1:
(11 comments)
http://gerrit.cloudera.org:8080/#/c/7438/1/be/src/exprs/expr-test.cc
File be/src
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
IMPALA-5513: Fix display message exception when using invalid KEYVAL
Fu
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 8: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7187
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 4:
(3 comments)
LGTM, have some small nits.
http://gerrit.cloudera.org:8080/#/c/7332/4//
anujphadke has posted comments on this change.
Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel
..
Patch Set 4:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7393/3/be/src/exec/partitioned-ha
Hello Thomas Tauber-Marshall,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7393
to look at the new patch set (#4).
Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel
..
Bikramjeet Vig has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7449
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
IMPALA-5275: Avoid printing status stack trace on hot pat
Taras Bobrovytsky has uploaded a new patch set (#2).
Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
..
IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
Implement the new DECIMAL return type rules for mult
Taras Bobrovytsky has uploaded a new patch set (#2).
Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
..
IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
Implement the new DECIMAL return type rules for mult
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-4939, IMPALA-4939: Decimal V2 multiplication
..
Patch Set 1:
(1 comment)
I still have to check how this patch impacts performance.
http://gerrit.clo
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 4: Code-Review+1
(2 comments)
I think it looks pretty good, please address the few small commen
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5987/2/tests/query_test/test_lifecycle.py
F
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5987/2/tests/query_test/test_lifecycle.py
Fi
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Rebase, carry +2.
--
To view, visit http://gerrit.cloudera.org:8080/7418
T
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 4:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 4: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7418/1/CMakeLi
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/7445
To unsubscribe, visit http://gerrit.c
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/1/be/src/runtime/exec-env.cc
File be/src/runtime/exe
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2:
Thanks for the ping - I think this still worth doing, and is an even easier fix
thanks to r
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5520: TopN node periodically reclaims old allocations
..
Patch Set 6:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/864/
Henry Robinson has uploaded a new patch set (#2).
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
IMPALA-5670: Misc. tidying of ExecEnv
* Remove duplicated c'tor code for ExecEnv() and delegate it onto
6-argument c'to
Henry Robinson has uploaded a new patch set (#2).
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
IMPALA-4925: Cancel finstance if query has finished
This patch is a partial fix for the issue where an fins
Michael Ho has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 4: Code-Review+1
(1 comment)
LGTM. Tim may want to do another pass.
http://gerrit
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7418/3/CMakeLists.txt
File
Michael Ho has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7418/3/CMakeLists.txt
File CMak
Michael Ho has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 2:
(2 comments)
Thanks for doing the fix. It looks good to me.
As discussed in person, please add a
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 8:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryr
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 8: Code-Review+2
Rebased and resubmitting GVO (failed due to a recently
Bharath Vissapragada has uploaded a new patch set (#3).
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:
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7418/3/CMakeLists.txt
File
Henry Robinson has uploaded a new patch set (#4).
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
IMPALA-5659: Begin standardizing treatment of thirdparty libraries
If Impala was built with
John Russell has posted comments on this change.
Change subject: [DOCS] Fix URL of link to glog project
..
Patch Set 1:
Adding Laurel as reviewer because she helped identify the outdated URL.
--
To view, visit http://gerrit.c
John Russell has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7447
Change subject: [DOCS] Fix URL of link to glog project
..
[DOCS] Fix URL of link to glog project
Formerly the project used SVN and the in
anujphadke has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 4:
(10 comments)
http://gerrit.cloudera.org:8080/#/c/7332/3//COMMIT_MSG
Commit Message:
PS3, Line
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:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/7416/2/fe/src/
Hello Matthew Jacobs,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7419
to look at the new patch set (#5).
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
...
Bikramjeet Vig has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 4:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7419/4/tests/query_tes
anujphadke has uploaded a new patch set (#4).
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 ca
Michael Ho has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7418/3/CMakeLists.txt
File CMa
Thomas Tauber-Marshall has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7446
Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables
..
IMPALA-5546: Allow creating unpartitioned Kudu tables
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and
IGNORE NULLS
..
Patch Set 2:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/7416/2/fe/src/main/j
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/1/be/src/runtime/exec-env.cc
File be/src/runtime/exec
Henry Robinson has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7445
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
IMPALA-5670: Misc. tidying of ExecEnv
* Remove duplicated c'tor code for ExecE
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5667: Race in DataStremSender could cause TransmitData
sidecar corruption
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7436/1/be/src/
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 7: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-v
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5520: TopN node periodically reclaims old allocations
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7400
To unsubscrib
Jim Apple 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 be/src/gu
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-5498: Support for partial sorts in Kudu INSERTs
..
Patch Set 6:
(14 comments)
http://gerrit.cloudera.org:8080/#/c/7267/6//COMMIT_MSG
Commit Mess
Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7267
to look at the new patch set (#7).
Change subject: IMPALA-5498: Support for partial sorts in Kudu INSERTs
..
IMPAL
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5667: Race in DataStremSender could cause TransmitData
sidecar corruption
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7436/1/be/src
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryr
Jim Apple has posted comments on this change.
Change subject: IMPALA-5513: Fix display message exception when using invalid
KEYVAL
..
Patch Set 7: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7187
To unsubs
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 4:
(2 comments)
looks good, just one possible test concern.
http://gerrit
Thomas Tauber-Marshall has submitted this change and it was merged.
Change subject: Bump Kudu version to f144d57
..
Bump Kudu version to f144d57
Change-Id: Ic9986f8dfaecef00e87e1cef93fc5e5e9dc8c810
---
M buildall.sh
1 file chan
Thomas Tauber-Marshall has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7443
To unsubscribe, visit http://gerrit.clou
Matthew Jacobs has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7443
To unsubscribe, visit http://gerrit.cloudera.
Thomas Tauber-Marshall has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7443
Change subject: Bump Kudu version to f144d57
..
Bump Kudu version to f144d57
Change-Id: Ic9986f8dfaecef00e87e1cef93fc5e5e9dc8c8
65 matches
Mail list logo
