Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
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: > > >

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Jim Apple
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

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Jim Apple
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

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada
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

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
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

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada
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? > >

Re: [Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Lars Volker
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-08-02 Thread Bharath Vissapragada (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-24 Thread Internal Jenkins (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-24 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-24 Thread Lars Volker (Code Review)
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 ..

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-23 Thread Lars Volker (Code Review)
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 ..

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-22 Thread Lars Volker (Code Review)
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 ..

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-22 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-21 Thread Marcel Kornacker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-21 Thread Lars Volker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-21 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-20 Thread Marcel Kornacker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-15 Thread Lars Volker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-15 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-12 Thread Marcel Kornacker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-07-11 Thread Marcel Kornacker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-06-06 Thread Lars Volker (Code Review)
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:

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-06-03 Thread Lars Volker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-12 Thread Marcel Kornacker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-11 Thread Marcel Kornacker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-11 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-10 Thread Lars Volker (Code Review)
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.

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-10 Thread Lars Volker (Code Review)
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

[Impala-CR](cdh5-trunk) IMPALA-2979: Fix scheduling on remote hosts

2016-05-10 Thread Lars Volker (Code Review)
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