Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 23:
This change did not cherrypick successfully
Tim Armstrong has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
IMPALA-5216: Make admission control queuing async
Impleme
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 22:
exhaustive tests passed too.
--
To view, visit ht
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 22: Verified+1
--
To view, visit http://gerrit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 22:
Build started: https://jenkins.impala.io/jo
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 22: Code-Review+2
Another silly test bug related to que
Tim Armstrong has uploaded a new patch set (#22) to the change originally
created by Bikramjeet Vig. ( http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
IMPALA-5216: Mak
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 21: Verified-1
Build failed: https://jenkins.im
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 21:
Build started: https://jenkins.impala.io/jo
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 21: Code-Review+2
--
To view, visit http://ger
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 20: Code-Review+2
--
To view, visit http://gerrit.cloudera
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 20:
Dan, it's probably worth looking at PS19->20. It's
Tim Armstrong has uploaded a new patch set (#20) to the change originally
created by Bikramjeet Vig. ( http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
IMPALA-5216: Mak
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 19:
So it turns out the test was already broken - the q
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 19:
Another clang-tidy error in promise-test and a flak
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 19: Verified-1
Build failed: https://jenkins.im
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 18:
Build started: https://jenkins.impala.io/jo
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 19: Code-Review+2
--
To view, visit http://ger
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 19:
Build started: https://jenkins.impala.io/jo
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 18: Code-Review+2
--
To view, visit http://gerrit.clou
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 17:
Fixed TSAN and clang-tidy compile warnings.
--
To
Tim Armstrong has uploaded a new patch set (#18) to the change originally
created by Bikramjeet Vig. ( http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
IMPALA-5216: Mak
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 17: Verified-1
Build failed: https://jenkins.im
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 17:
Build started: https://jenkins.impala.io/jo
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 17: Code-Review+2
--
To view, visit http://ger
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 16:
Build started: https://jenkins.impala.io/jo
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 16: Code-Review+1
--
To view, visit http://gerrit.clou
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 16: Code-Review+2
Tim, do you have any more comments?
--
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 16:
> - GetOperationStatus() after rejection should show re
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 15:
> Could you file a JIRA for that? We probably have
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#16).
Change subject: IMPALA-5216: Make admission control queuing async
..
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 15:
> (2 comments)
>
> > Do we have sufficient test cover
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#15).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 14:
(2 comments)
> Do we have sufficient test coverag
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 14:
(1 comment)
Just rebased in patchset 14, will add
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#14).
Change subject: IMPALA-5216: Make admission control queuing async
..
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 12:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/1
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 12:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10060/
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#13).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 12:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 12:
(5 comments)
Looks nice. Please go ahead and rebase wh
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#12).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 11:
(30 comments)
http://gerrit.cloudera.org:8080/#/c
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 11:
(29 comments)
That looks good. My review comments are
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#11).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 10:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/1
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 10:
> Uploaded patch set 10.
No notable conflicts dur
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#10).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 9:
> > (2 comments)
> >
> > will rebase in next patc
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 9:
> (2 comments)
>
> will rebase in next patch, please h
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 9:
(2 comments)
will rebase in next patch, please hol
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#9).
Change subject: IMPALA-5216: Make admission control queuing async
..
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10060/7
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 8:
(3 comments)
Just wanted to get the replay regardi
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/10060/
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10060/7
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 8:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(40 comments)
http://gerrit.cloudera.org:8080/#/c/
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#8).
Change subject: IMPALA-5216: Make admission control queuing async
..
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(8 comments)
I think these are my final comments for th
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/10060/7
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(5 comments)
Will have some more comments but wanted to
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(27 comments)
I need to dig into the details some more
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 7:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/10
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#7).
Change subject: IMPALA-5216: Make admission control queuing async
..
Bikramjeet Vig has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 5:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/1
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#6).
Change subject: IMPALA-5216: Make admission control queuing async
..
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/10060 )
Change subject: IMPALA-5216: Make admission control queuing async
..
Patch Set 5:
(5 comments)
I'm ready to +1 after these issues and
Hello Tim Armstrong, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10060
to look at the new patch set (#5).
Change subject: IMPALA-5216: Make admission control queuing async
..
70 matches
Mail list logo