Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 5:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/878/
--
To view, visit
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/4/be/src/testutil/in-process-servers.h
File
Hello 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 (#5).
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Michael Ho has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 4: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/4/be/src/testutil/in-process-servers.h
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2: Code-Review+2
Included patch for IMPALA-5670 broke some tests. Rebased to remove it, carry
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 3:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/877/
--
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/874/
Hello 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 (#4).
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
IMPALA-5670: Misc.
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 3:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/7445/3/be/src/testutil/in-process-servers.h
File
Impala Public Jenkins 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/7451
To unsubscribe, visit
Impala Public Jenkins has submitted this change and it was merged.
Change subject: Bump Kudu version to f144d57
..
Bump Kudu version to f144d57
Change-Id: Iea7d94f9807fade7dca781814d795265e254b20e
Reviewed-on:
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: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/876/
--
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 (#24).
Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build.
Henry Robinson has submitted this change and it was merged.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
IMPALA-5659: Begin standardizing treatment of thirdparty libraries
If Impala
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 3:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/875/
--
To
Michael Ho has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7394
To unsubscribe, visit
Michael Ho has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 3:
(3 comments)
Thanks for cleaning things up. Looks much better.
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3: Verified-1
Build failed:
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7418
To
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/7418
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: Verified-1
Build failed:
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 1: Verified-1
Build failed:
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/874/
--
To
Tim Armstrong has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7459
Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2
..
IMPALA-5676: avoid expensive consistency checks in BTSv2
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2: Code-Review+2
Oops I mean to +2
--
To view, visit http://gerrit.cloudera.org:8080/5987
To
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4674: add new message to stress test
..
Patch Set 1:
Dan Hecht already reviewed this as part of the big patch
https://gerrit.cloudera.org/#/c/5801/, but
Tim Armstrong has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7458
Change subject: IMPALA-4674: add new message to stress test
..
IMPALA-4674: add new message to stress test
The main IMPALA-4674 commit
Impala Public Jenkins has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/873/
--
To view, visit
Thomas Tauber-Marshall has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1: Code-Review+2
The GVO failure is definitely unrelated, as the test has nothing to do with
Kudu. I filed:
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4925: Cancel finstance if query has finished
..
Patch Set 2:
Thanks for the review - any other comments?
--
To view, visit
Michael Brown has posted comments on this change.
Change subject: [DOCS] Update list of reserved words
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7452
To unsubscribe, visit
Henry Robinson has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7457
Change subject: IMPALA-4905: Don't send empty insert status to coordinator
..
IMPALA-4905: Don't send empty insert status to
Impala Public Jenkins has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/868/
--
To view, visit
Matthew Mulder has uploaded a new patch set (#2).
Change subject: IMPALA-5625: write profile when query times out
..
IMPALA-5625: write profile when query times out
This change writes query profiles as text files for all of the
Bikramjeet Vig has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
(2 comments)
> I think we should do this for MemLimitExceeded() too:
>
> Joe
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
(2 comments)
Hello 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 (#3).
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
IMPALA-5670: Misc.
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7445/2/be/src/runtime/exec-env.cc
File
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/872/
Michael Ho has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7445/2/be/src/runtime/exec-env.cc
File
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
(2 comments)
This should be a good improvement! Just have a couple of comments.
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6898
To
Tim Armstrong 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)
LGTM aside from expanding the matrix of query options
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 3:
(1 comment)
Thomas Tauber-Marshall has uploaded a new patch set (#3).
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
IMPALA-5167: Reduce the number of Kudu clients created (FE)
Creating Kudu clients is very
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5670: Misc. tidying of ExecEnv
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/7445/2/be/src/runtime/exec-env.cc
File
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
I think we should do this for MemLimitExceeded() too:
Joe pointed out on
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5511: Add process start time to debug web page
..
Patch Set 6:
(6 comments)
I think the code would be cleaner if the metric always had the same name -
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 2:
(1 comment)
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:
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths
..
Patch Set 1:
(2 comments)
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4276: Profile displays non-default query options set by
planner
..
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7419
Henry Robinson has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 6:
Fix a bug in the way HDFS libraries were linked for libfesupport.so (must
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 6:
Build started:
Hello Impala Public Jenkins, Michael Ho, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7418
to look at the new patch set (#6).
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 1:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/869/
Matthew Jacobs has posted comments on this change.
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/7454
To
Matthew Jacobs has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7454
Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies
..
IMPALA-5638: Fix Kudu table set tblproperties
Attila Jeges has uploaded a new patch set (#3).
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
Attila Jeges has posted comments on this change.
Change subject: IMPALA-5407: Fix crash in HdfsSequenceTableWriter
..
Patch Set 2:
(2 comments)
Added a test case to seq-writer.test and made sure that it crashes impalad if
Attila Jeges has uploaded a new patch set (#3).
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
John Russell has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7452
Change subject: [DOCS] Update list of reserved words
..
[DOCS] Update list of reserved words
For 2.9, I believe the only new reserved
Michael Brown 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.
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Patch Set 6: Code-Review+1
(1 comment)
Not sure if can +2 this given the change isn't trivial, so
Impala Public Jenkins has posted comments on this change.
Change subject: Bump Kudu version to f144d57
..
Patch Set 1:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/868/
--
To view, visit
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/7451
To unsubscribe, visit
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
Kim Jin Chul has posted comments on this change.
Change subject: IMPALA-5529: Add additional function signatures for TRUNC()
..
Patch Set 1:
(4 comments)
Hi,
Would you please review my code with comments? Thanks.
I would
Gabor Kaszab has uploaded a new patch set (#6).
Change subject: IMPALA-5511: Add process start time to debug web page
..
IMPALA-5511: Add process start time to debug web page
Read the start date and time of the impalad,
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool
..
Patch Set 33: Code-Review+2
Fixed a bug in the PHJ debug string where it didn't handle a closed build
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 (#33).
Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool
..
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
Patch Set 2:
(1 comment)
Thomas Tauber-Marshall has uploaded a new patch set (#2).
Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE)
..
IMPALA-5167: Reduce the number of Kudu clients created (FE)
Creating Kudu clients is very
Thomas Tauber-Marshall has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7451
Change subject: Bump Kudu version to f144d57
..
Bump Kudu version to f144d57
Change-Id:
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 (#6).
Change subject: IMPALA-5582: Store sentry privileges in lower case
..
Kim Jin Chul has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7450
Change subject: IMPALA-5529: Add additional function signatures for TRUNC()
..
IMPALA-5529: Add additional function signatures for
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Build failed:
77 matches
Mail list logo
