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: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun
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 writer
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, visi
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 http
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 be/src/t
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
..
IMPALA-567
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
Fil
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 vi
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. tid
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 be/src/
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 http://gerrit.cloud
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: http://gerrit.clo
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 was
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 view
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 http:/
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.
http://gerrit.cloudera.org:8080/#/c/7445/3/be
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: http://jenkins.impala.io:8080/job/gerrit-verify-dryr
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 unsu
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
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 5: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit
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: http://jenkins.impala.io:8080/job/gerrit-verify-dr
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 vi
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
Doi
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 u
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 a
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 http://gerrit
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: I
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 http://gerrit.cloudera.org:8
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 http://gerrit.cl
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 coordinator
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 htt
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 poi
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)
http://gerrit.cloudera.org:8080/#/c/7449/1/be/src/service/impala-server
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. tid
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 be/src/runtime/ex
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 be/src/runtime/exec-e
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.
htt
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 unsubscribe
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
http://gerrit
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)
http://gerrit.cloudera.org:8080/#/c/6898/2/fe/src/main/java/org
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 e
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 be/src/runtime/exe
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 IMPALA-5598
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 - d
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)
http://gerrit.cloudera.org:8080/#/c/6898/2/fe/src/main/java/org/apache/
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: http://jenkins.impala.io:8080/job/gerrit-verify-dr
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)
http://gerrit.cloudera.org:8080/#/c/7449/1/be/src/service/impala-serve
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
To
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
a
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 6:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun
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 unsubscri
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 inconsist
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 crash
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
the
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 crash
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
key
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.
http://gerrit.cl
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 http://gerrit
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 http://gerrit.cloudera.
Jim Apple 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 be/src/gut
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 be/src
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 like
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, catalogd
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
pa
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
..
IMPALA-467
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)
http://gerrit.cloudera.org:8080/#/c/6898/1/fe/src/main/java/org
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 e
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: Iea7d94f9807fade7dca781814d795265e254b2
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
..
IMPALA-5
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 TRUNC()
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5659: Begin standardizing treatment of thirdparty
libraries
..
Patch Set 5:
Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/
81 matches
Mail list logo
