Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
IMPALA-9979: part 2: partitioned top-n
Planner changes:
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 35: Verified+1
--
To view, visit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 35: Code-Review+2
--
To view, visit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 35:
Build started:
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 34: Code-Review+2
--
To view, visit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 34:
Build Successful
Hello Aman Sinha, Qifan Chen, Thomas Tauber-Marshall, Shant Hovsepian, David
Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#34).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 33:
(2 comments)
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 33:
(3 comments)
Just one other comment after which I am ready to
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
(21 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 32:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 31:
Build Successful
Hello Aman Sinha, Qifan Chen, Thomas Tauber-Marshall, Shant Hovsepian, David
Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#32).
Change subject: IMPALA-9979: part 2: partitioned top-n
Hello Aman Sinha, Qifan Chen, Thomas Tauber-Marshall, Shant Hovsepian, David
Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#31).
Change subject: IMPALA-9979: part 2: partitioned top-n
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30: Code-Review+1
With the exception of small number of review
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
(5 comments)
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
(15 comments)
This is a complex optimization. Nice work on
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
> Patch Set 30: Code-Review+1
>
> BE part and tests look good to
Thomas Tauber-Marshall has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30: Code-Review+1
BE part and tests look good to me
--
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 30:
(1 comment)
Hello Aman Sinha, Qifan Chen, Thomas Tauber-Marshall, Shant Hovsepian, David
Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#30).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 29:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 29:
(1 comment)
Hello Aman Sinha, Qifan Chen, Thomas Tauber-Marshall, Shant Hovsepian, David
Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#29).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 28:
(7 comments)
Thomas Tauber-Marshall has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 28:
(7 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 28:
Build Successful
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#28).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 27: Verified+1
--
To view, visit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 27:
Build Successful
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 27:
Rebased onto master after the previous patch was merged
--
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 27:
Build started:
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#27).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 26:
Build Successful
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#26).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 25:
Latest patchset fixes the test failure and is ready for
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 24: Verified-1
Build failed:
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 25:
Build Successful
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#25).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 24:
Build started:
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 23: Verified-1
Build failed:
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 24:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 23:
Build Successful
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#24).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 23:
Build started:
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 23:
(2 comments)
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#23).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 22:
(1 comment)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 22:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 21:
Build Successful
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#22).
Change subject: IMPALA-9979: part 2: partitioned top-n
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#21).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 21:
(4 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 20:
Build Failed
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 20:
(1 comment)
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 19:
(1 comment)
Just another WIP, but I made some progress
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#20).
Change subject: IMPALA-9979: part 2: partitioned top-n
Qifan Chen has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 19: Code-Review+1
(8 comments)
Looks good and thanks a lot for the
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 19:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 19:
(1 comment)
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 19:
This is the rebased version, using the new PriorityQueue
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#19).
Change subject: IMPALA-9979: part 2: partitioned top-n
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 18:
Build Successful
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 18:
(1 comment)
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(27 comments)
Pushing out the changes I've done so far. I'm
Hello Aman Sinha, Qifan Chen, Shant Hovsepian, David Rorke, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#18).
Change subject: IMPALA-9979: part 2: partitioned top-n
Shant Hovsepian has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17: Code-Review+1
(2 comments)
I don't feel as strongly about
Qifan Chen has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(5 comments)
Looks very good!
Aman Sinha has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(5 comments)
Sending initial comments. The patch looks in
Qifan Chen has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(8 comments)
1st batch of comments. Will try to find some time
Shant Hovsepian has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(1 comment)
Shant Hovsepian has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
(6 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 17:
Build Successful
Hello Aman Sinha, Shant Hovsepian, David Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#17).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 15:
(3 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 16:
Build Successful
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 15:
(3 comments)
This is ready for review. I still need to look
Hello Aman Sinha, Shant Hovsepian, David Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#16).
Change subject: IMPALA-9979: part 2: partitioned top-n
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 12:
(2 comments)
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/16242 )
Change subject: IMPALA-9979: part 2: partitioned top-n
..
Patch Set 12:
Build Successful
Hello Aman Sinha, Shant Hovsepian, David Rorke, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16242
to look at the new patch set (#12).
Change subject: IMPALA-9979: part 2: partitioned top-n
82 matches
Mail list logo