Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 17:
(4 comments)
Please see my inline comments and PS17.
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#17).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#16).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 8:
Build started: https://jenkins.impala.io/job/
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 8:
Failing in "Waiting for HDFS replication"
--
To vie
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1:
Build started: https://jenkins.impala.io/j
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1:
GVO failed due to infrastructure issue (IMPALA-6394)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1: Verified-1
Build failed: https://jenkins.i
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
Patch Set 8: Verified+1
--
To vi
Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
IMPALA-5518: Allocate Krpc
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 8: Verified-1
Build failed: https://jenkins.impa
Hello Thomas Tauber-Marshall, Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9306
to look at the new patch set (#9).
Change subject: IMPALA-5752: Add support for DECIMAL on Kudu tables
.
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 15: Code-Review+2
(5 comments)
http://gerrit.cloudera.org:
Sailesh Mukil has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 15: Code-Review+1
(2 comments)
http://gerrit.cloudera.o
Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 14:
(4 comments)
Thanks for the review. Please see PS15 a
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#15).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Sailesh Mukil has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 14: Code-Review+1
(4 comments)
http://gerrit.cloudera.o
Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
IMPALA-6423: HDFS scanner doesn
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
Patch Set 3: Verified+1
--
To view, v
Hello Philip Zeyliger, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9227
to look at the new patch set (#6).
Change subject: IMPALA-6482: add QUERY_TIME_LIMIT_S option
..
IMPAL
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9227 )
Change subject: IMPALA-6482: add QUERY_TIME_LIMIT_S option
..
Patch Set 5:
(2 comments)
I also considered calling this something like
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1:
Build started: https://jenkins.impala.io/j
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
Patch Set 8:
Build started: https
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
Patch Set 8: Code-Review+2
Carry Tim's +2
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
Patch Set 8:
(2 comments)
http://gerrit.clo
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9344
to look at the new patch set (#8).
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
...
Sailesh Mukil has posted comments on this change. (
http://gerrit.cloudera.org:8080/9361 )
Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors
in Writev
..
Patch Set 1: Code-Review+2
--
To view, vi
Sailesh Mukil has posted comments on this change. (
http://gerrit.cloudera.org:8080/9360 )
Change subject: [security] test and fixes for TLS socket EINTR issues
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cl
Sailesh Mukil has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.clo
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 8:
Build started: https://jenkins.impala.io/job/
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 8: Code-Review+2
--
To view, visit http://gerrit.cloude
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9227 )
Change subject: IMPALA-6482: add QUERY_TIME_LIMIT_S option
..
Patch Set 5:
(2 comments)
I just looked at how these two were differen
Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 7: Code-Review+2
--
To view, visit http://gerrit.cloudera
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9271 )
Change subject: IMPALA-6497: add "Last row fetched" and AC events
..
Patch Set 7:
Any more comments?
--
To view, visit http://gerrit.
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 9: Code-Review+2
(2 comments)
This seems like an improv
Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 13:
(6 comments)
PS13 contains the rebase. PS14 should ad
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#14).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#13).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Patch Set 1:
Clean application of the original patch.
--
To view
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9361 )
Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors
in Writev
..
Patch Set 1:
Clean application of the origina
Michael Ho has removed Kudu Jenkins from this change. (
http://gerrit.cloudera.org:8080/9361 )
Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors
in Writev
..
Removed reviewer Kudu Jenkins.
--
To v
Michael Ho has posted comments on this change. (
http://gerrit.cloudera.org:8080/9360 )
Change subject: [security] test and fixes for TLS socket EINTR issues
..
Patch Set 1:
Clean application of the original patch.
--
To vie
Michael Ho has removed Kudu Jenkins from this change. (
http://gerrit.cloudera.org:8080/9360 )
Change subject: [security] test and fixes for TLS socket EINTR issues
..
Removed reviewer Kudu Jenkins.
--
To view, visit http://ge
Hello Alexey Serbin, Kudu Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9361
to review the following change.
Change subject: KUDU-2218. tls_socket: properly handle temporary socket errors
in Writev
.
Hello Mike Percy, Kudu Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9359
to review the following change.
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
K
Michael Ho has removed Kudu Jenkins from this change. (
http://gerrit.cloudera.org:8080/9359 )
Change subject: KUDU-2004. Undefined behavior in TlsSocket::Writev()
..
Removed reviewer Kudu Jenkins.
--
To view, visit http://ger
Hello Alexey Serbin, Dan Burkert, Kudu Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9360
to review the following change.
Change subject: [security] test and fixes for TLS socket EINTR issues
Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9292 )
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Patch Set 11:
(11 comments)
Thanks for the review. Please see PS11
Hello Michael Ho, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9292
to look at the new patch set (#12).
Change subject: IMPALA-6269: Expose KRPC metrics on debug webpage
..
Lars Volker has posted comments on this change. (
http://gerrit.cloudera.org:8080/9358 )
Change subject: IMPALA-6538: Fix read path when Parquet min/max statistics
contain NaN
..
Patch Set 2:
(2 comments)
http://gerrit.cloud
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9344 )
Change subject: IMPALA-5518: Allocate KrpcDataStreamRecvr RowBatch tuples from
BufferPool
..
Patch Set 7: Code-Review+2
(2 comments)
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
Patch Set 3: Code-Review+2
--
To view, visit
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
Patch Set 2: Code-Review+2
--
To view, visit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
Patch Set 3:
Build started: https://je
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9358 )
Change subject: IMPALA-6538: Fix read path when Parquet min/max statistics
contain NaN
..
Patch Set 2:
(1 comment)
http://gerrit.clou
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8966 )
Change subject: IMPALA-4835: Part 3: switch I/O buffers to buffer pool
..
Patch Set 17:
Rebased and fixed a couple of bugs with reserva
Hello Bikramjeet Vig,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8966
to look at the new patch set (#17).
Change subject: IMPALA-4835: Part 3: switch I/O buffers to buffer pool
..
IMPA
Hello Thomas Tauber-Marshall, Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9306
to look at the new patch set (#8).
Change subject: IMPALA-5752: Add support for DECIMAL on Kudu tables
.
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 28:
(1 comment)
http://gerrit.cloudera.org:8080/
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 31: Code-Review+2
--
To view, visit http://gerri
Hello Michael Ho, Tianyi Wang, Bikramjeet Vig, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8707
to look at the new patch set (#31).
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8414 )
Change subject: IMPALA-4835: Part 1: simplify I/O mgr mem mgmt and cancellation
..
Patch Set 23: Code-Review+2
--
To view, visit http:
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 29: Code-Review+2
--
To view, visit http://gerri
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 30: Code-Review+2
--
To view, visit http://gerri
Hello Michael Ho, Tianyi Wang, Bikramjeet Vig, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8707
to look at the new patch set (#29).
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 28: Code-Review+2
Carry +2
--
To view, visit ht
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8707 )
Change subject: IMPALA-4835: Part 2: Allocate scan range buffers upfront
..
Patch Set 28:
(3 comments)
http://gerrit.cloudera.org:8080
Hello Thomas Tauber-Marshall, Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9306
to look at the new patch set (#7).
Change subject: IMPALA-5752: Add support for DECIMAL on Kudu tables
.
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 9:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/9063/
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9358 )
Change subject: IMPALA-6538: Fix read path when Parquet min/max statistics
contain NaN
..
Patch Set 2:
(2 comments)
Seems fine in gene
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9352 )
Change subject: IMPALA-6423: HDFS scanner doesn't check
RuntimeState::is_cancelled()
..
Patch Set 2: Code-Review+1
--
To view, visit h
Hello Gabor Kaszab, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9358
to look at the new patch set (#2).
Change subject: IMPALA-6538: Fix read path when Parquet min/max statistics
contain NaN
...
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/90
Zoltan Borok-Nagy has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9358
Change subject: IMPALA-6538: Fix read path when Parquet min/max statistics
contain NaN
..
IMPALA-6538: Fix read path when Parquet mi
Hello Thomas Tauber-Marshall, Taras Bobrovytsky,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9306
to look at the new patch set (#6).
Change subject: IMPALA-5752: Add support for DECIMAL on Kudu tables
.
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 9:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/9
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/9063/
Hello Lars Volker, Laszlo Gaal, Gabor Kaszab, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9063
to look at the new patch set (#9).
Change subject: IMPALA-5801: Clean up codegen GetType() interface
...
Laszlo Gaal has posted comments on this change. (
http://gerrit.cloudera.org:8080/9063 )
Change subject: IMPALA-5801: Clean up codegen GetType() interface
..
Patch Set 8:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/9063/
Hello Lars Volker, Dimitris Tsirogiannis, Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8523
to look at the new patch set (#11).
Change subject: IMPALA-5931: Generates scan ranges in planner for s3/adls
...
80 matches
Mail list logo