Tim Armstrong has uploaded a new patch set (#2).
Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2
..
IMPALA-5676: avoid expensive consistency checks in BTSv2
Doing an O(n) consistency check every time
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2
..
Patch Set 1:
(1 comment)
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
Patch Set 2:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/893/
--
To
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
Patch Set 2: Code-Review+2
rebase
--
To view, visit http://gerrit.cloudera.org:8080/7466
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 4:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/892/
--
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 4: Code-Review+2
rebase
--
To view, visit http://gerrit.cloudera.org:8080/7454
To
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
IMPALA-5686: Update a mini cluster Sentry property
The property
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7469
To unsubscribe,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build.
..
Patch Set 24:
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/890/
--
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 3: Verified-1
Build failed:
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
Patch Set 1: Verified-1
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/888/
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 8: Verified-1
Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/887/
--
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 2:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/891/
--
To view,
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 2: Code-Review+2
Forwarding the +2. Thanks for the review, Henry and Lars.
--
To view,
Taras Bobrovytsky has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 1:
(1 comment)
Yes, I tested this change on my machine and it fixed the issue for me.
Lars Volker has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 1: Code-Review+2
I cherry picked the change and it fixed the issue for me.
--
To view, visit
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build.
..
Patch Set 24:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/890/
--
To
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
Patch Set 1: Code-Review+2
(1 comment)
Did this fix the issue for you?
Taras Bobrovytsky has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7469
Change subject: IMPALA-5686: Update a mini cluster Sentry property
..
IMPALA-5686: Update a mini cluster Sentry property
The
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 3:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/889/
--
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 3: Code-Review+2
rebase
--
To view, visit http://gerrit.cloudera.org:8080/7454
To
Henry Robinson has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7467
Change subject: (PREVIEW) IMPALA-5684: Optionally run be tests in sharded mode
..
(PREVIEW) IMPALA-5684: Optionally run be tests in
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
Patch Set 1:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/888/
--
To
David Knupp has posted comments on this change.
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7466
To unsubscribe, visit
Michael Brown has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 8: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/7332
To unsubscribe, visit
Matthew Jacobs has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7466
Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test
..
IMPALA-4868: Disable flaky TestRequestPoolService test
Michael Brown has posted comments on this change.
Change subject: IMPALA-5625: write profile when query times out
..
Patch Set 2:
(8 comments)
Thanks for this patch, I think this will be a great improvement.
Henry Robinson has submitted this change and it was merged.
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
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 6: Code-Review+2
Fix typo, carry +2.
--
To view, visit http://gerrit.cloudera.org:8080/7445
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7445
To unsubscribe, visit
anujphadke has posted comments on this change.
Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel
..
Patch Set 5:
(1 comment)
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 (#5).
Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
The
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7311
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 6: Verified-1
Build failed:
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 2: Code-Review+1
(1 comment)
Looks good, this should be a big improvement. There's an
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 2: Verified-1
Build failed:
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 2: Code-Review+1
Looks good to me. Lets give Tim and/or Sailesh another chance to
Bikramjeet Vig has uploaded a new patch set (#2).
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
IMPALA-5275: Avoid printing status stack trace on hot paths
Currently, creation of a Status object
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 8:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/887/
--
To view,
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 4:
(1 comment)
Hello Impala Public Jenkins, Matthew Jacobs,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6898
to look at the new patch set (#4).
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 8: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7332
To unsubscribe, visit
anujphadke has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 8:
(2 comments)
Hello Bharath Vissapragada, 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 (#8).
Change subject: IMPALA-5582: Store sentry privileges in lower case
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 6:
Build started:
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7419
Hello Impala Public Jenkins, Henry Robinson, 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 (#6).
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 7:
(2 comments)
Thanks! 2 small things and otherwise I'm good to go. Let's make sure Michael
anujphadke has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 7:
(4 comments)
Hello Bharath Vissapragada, 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 (#7).
Change subject: IMPALA-5582: Store sentry privileges in lower case
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3:
> So the GVO failed because there's a custom cluster test that runs
> impala with
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3:
So the GVO failed because there's a custom cluster test that runs impala
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7454
To
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4086: Add benchmark for simple scheduler
..
Patch Set 5:
There's no rush, we can leave it open, just wanted to clarify what the state of
it was.
--
To
Lars Volker has posted comments on this change.
Change subject: IMPALA-4086: Add benchmark for simple scheduler
..
Patch Set 5:
> Did this get lost in the shuffle? I'm trying to pare down my review
> inbox a bit :)
It's
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 6:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/885/
--
To view, visit
Hello Impala Public Jenkins, Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7445
to look at the new patch set (#6).
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 6:
> (4 comments)
>
> None of these comments are must-dos, just things I've noticed.
Anuj,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 5:
Build started:
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool
..
Patch Set 34: Code-Review+2
Updated test_explain with a minor change.
--
To view, visit
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 (#34).
Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool
..
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel
..
Patch Set 4:
(1 comment)
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4086: Add benchmark for simple scheduler
..
Patch Set 5:
Did this get lost in the shuffle? I'm trying to pare down my review inbox a bit
:)
--
To
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5317: add DATE_TRUNC() function
..
Patch Set 1:
Any updates here? Can I help move this along somehow?
--
To view, visit
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
Yes I think never printing stacktraces for MemLimitExceeded() makes sense. We
have
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
Patch Set 6:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/882/
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 2:
Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/881/
--
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 5:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/5/be/src/testutil/mini-impala-cluster.cc
File
Michael Ho has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 5:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/5/be/src/testutil/mini-impala-cluster.cc
File
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5987
To unsubscribe, visit
Henry Robinson has submitted this change and it was merged.
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
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7311
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
Patch Set 6:
Build started:
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
Patch Set 4:
(2 comments)
Matthew Jacobs has uploaded a new patch set (#6).
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
The
Matthew Jacobs has uploaded a new patch set (#5).
Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
..
IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts
The
Tim Armstrong has submitted this change and it was merged.
Change subject: IMPALA-4674: add new message to stress test
..
IMPALA-4674: add new message to stress test
The main IMPALA-4674 commit adds a new OOM failure mode
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4674: add new message to stress test
..
Patch Set 1: Verified+1
This isn't exercised by the pre-merge tests so will manually verify.
--
To view, visit
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 2:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/879/
Michael Brown has posted comments on this change.
Change subject: IMPALA-5625: write profile when query times out
..
Patch Set 2:
> I replied to them with "Done", but they show up as Draft, so I
> wonder if you don't see them.
Matthew Mulder has posted comments on this change.
Change subject: IMPALA-5625: write profile when query times out
..
Patch Set 1:
(8 comments)
> Matt, you uploaded a new patch set, but please reply to the inline
> comments
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7454
To
Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7454
to look at the new patch set (#2).
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
Michael Brown has posted comments on this change.
Change subject: IMPALA-5625: write profile when query times out
..
Patch Set 2:
Matt, you uploaded a new patch set, but please reply to the inline comments
with either "done"
Michael Brown has posted comments on this change.
Change subject: IMPALA-4674: add new message to stress test
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7458
To unsubscribe, visit
Attila Jeges has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 4:
> Patch Set 3: Code-Review+2
Thanks for reviewing it.
--
To view, visit
Attila Jeges has abandoned this change.
Change subject: IMPALA-3548: Prune runtime filters based on query options in
the FE
..
Abandoned
It was decided that another approach should be taken to implement this feature.
I'm
Kim Jin Chul has posted comments on this change.
Change subject: IMPALA-5116: Remove deprecated hash_* types in gutil
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7414/1/be/src/gutil/hash/hash.h
File
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5987
To unsubscribe,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 5: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/878/
--
To view,
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build.
..
Patch Set 24: Verified-1
Build failed:
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
IMPALA-5407: Fix crash in HdfsSequenceTableWriter
The following use of sequence file
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7394
To unsubscribe,
94 matches
Mail list logo