Just to clarify, this change has been moved to Impala-ASF and merged
successfully.
On Aug 2, 2016 18:30, "Jim Apple" wrote:
> For outdated changes, please use the instructions here:
>
>
>
Please do not commit to the "Impala" gerrit project any more. It is
stale and it will remain stale.
Use "Impala-ASF" for asf and use private jenkins for private github.
Public github (corresponding to public gerrit project "Impala") should
be stale at the moment. Please see our branching document
For outdated changes, please use the instructions here:
http://www.google.com/url?q=http%3A%2F%2Fmail-archives.apache.org%2Fmod_mbox%2Fincubator-impala-dev%2F201607.mbox%2F%253CCAC-pSX2sHyqsoxGCCKrqmQbo-jjWdP6CMGoTU_1snuQVkDVg_w%40mail.gmail.com%253E=D=1=AFQjCNFyYa3BsqALZuEvxDzgAV5l4P_evQ
"For
This commit has been cherry-picked like any other commit [1]. I pushed it
to gerrit bypassing the code review. Per my understanding, gerrit checks
the "Change-Id" field and sees if there is an existing review for that id
and branch and sends out a message. Once it is committed to the gerrit
I'm not sure at all what's supposed to happen with outdated changes in
cdh5-trunk. Probably this one should have been transported just as any
other change merged into asf/master. Maybe your transport did the right
thing and Gerrit picked it up wrongly? Did you sync git repositories or
Gerrit
Oops. I was sync'ing gerrit/cdh5-trunk and asf/master. They became off-sync
after the move to asf. Shouldn't this be cherry-picked ?
On Tue, Aug 2, 2016 at 5:27 PM, Lars Volker wrote:
> Hi Bharath,
>
> Did you mean to merge this? Or did I misinterpret the Gerrit mail?
>
>
Hi Bharath,
Did you mean to merge this? Or did I misinterpret the Gerrit mail?
Cheers, Lars
On Aug 2, 2016 12:55, "Bharath Vissapragada (Code Review)" <
ger...@cloudera.org> wrote:
> Bharath Vissapragada has submitted this change and it was merged.
>
> Change subject: IMPALA-2979: Fix
Bharath Vissapragada has submitted this change and it was merged.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
IMPALA-2979: Fix scheduling on remote hosts
Also fixes: IMPALA-2400, IMPALA-3043
This change
Hello Marcel Kornacker, Lars Volker, Internal Jenkins, Sailesh Mukil,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2200
to look at the new patch set (#33).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
Internal Jenkins has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 32: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/2200
To unsubscribe, visit
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 31: Code-Review+2
+2 from Marcel.
Fixed small errors related to tests running on a localhost multi-node
Hello Marcel Kornacker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2200
to look at the new patch set (#31).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Hello Marcel Kornacker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2200
to look at the new patch set (#30).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Hello Marcel Kornacker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2200
to look at the new patch set (#29).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 27:
(3 comments)
Thanks for the comments. I added more comments to the code but otherwise didn't
change
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 28: Code-Review+2
(3 comments)
let's get this in, and then think about additional test coverage.
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 27:
(15 comments)
Thanks for the review, please see PS28.
Lars Volker has uploaded a new patch set (#28).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
IMPALA-2979: Fix scheduling on remote hosts
Also fixes: IMPALA-2400, IMPALA-3043
This change fixes scheduling
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 27:
(8 comments)
http://gerrit.cloudera.org:8080/#/c/2200/24/be/src/scheduling/simple-scheduler.cc
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 24:
(55 comments)
Thanks for the review. Please see PS25.
Lars Volker has uploaded a new patch set (#25).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
IMPALA-2979: Fix scheduling on remote hosts
Also fixes: IMPALA-2400, IMPALA-3043
This change fixes scheduling
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 24:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/2200/24/be/src/scheduling/simple-scheduler.cc
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 24:
(49 comments)
http://gerrit.cloudera.org:8080/#/c/2200/2/be/src/scheduling/simple-scheduler.cc
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 23:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/2200/23/be/CMakeLists.txt
File be/CMakeLists.txt:
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 22:
(40 comments)
Thanks for the review. Please see PS23.
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 22:
(28 comments)
some of these comments were written prior to our conversation at 5pm today.
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 22:
(12 comments)
http://gerrit.cloudera.org:8080/#/c/2200/22/be/src/scheduling/simple-scheduler.h
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 22:
I noticed that this change needs to add test exercising different values of the
replica_preference
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 19:
(26 comments)
Thank you for the review. I addressed your comments, please see PS 22.
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 22: -Code-Review
Up for review. Please note that I had to rebase the change to get it on top of
the new
Lars Volker has uploaded a new patch set (#22).
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
IMPALA-2979: Fix scheduling on remote hosts
Also fixes: IMPALA-2400, IMPALA-3019, IMPALA-3043
This change fixes
31 matches
Mail list logo