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:
>
>
> http://www.google.com/url?q=http%3A%2F%2Fmail-archives.apache.org%2Fmod_mbox%2Fincubator-impala-dev%2F201607
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&sa=D&sntz=1&usg=AFQjCNFyYa3BsqALZuEvxDzgAV5l4P_
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
remote,
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 instan
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?
>
> Cheers, Lars
>
> On Aug
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 schedulin
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 fixe
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 http://g
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 c
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-2979:
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-2979:
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-2979:
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.
ht
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.
http://gerrit.cloudera.org:8080/#/c/2200/24/be
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 scan-r
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
F
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
F
Lars Volker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 25:
(20 comments)
Thanks for the review, please see PS27.
http://gerrit.cloudera.org:8080/#/c/2200/24/be
Lars Volker has uploaded a new patch set (#27).
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 scan-r
Lars Volker has uploaded a new patch set (#26).
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 scan-r
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 25:
(20 comments)
the header file looks good structurally
http://gerrit.cloudera.org:8080/#/c/2200/
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.
http://gerrit.cloudera.org:8080/#/c/2200/24/be
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 scan-r
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
F
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
F
Sailesh Mukil has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 24:
(3 comments)
Just passing through.
http://gerrit.cloudera.org:8080/#/c/2200/24/be/src/scheduling/s
Lars Volker has uploaded a new patch set (#24).
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 scan-r
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:
Li
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.
http://gerrit.cloudera.org:8080/#/c/2200/22/be
Lars Volker has uploaded a new patch set (#23).
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 scan-r
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.
ignor
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-2979: Fix scheduling on remote hosts
..
Patch Set 19:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/2200/19/be/src/scheduling/simple-scheduler.h
Fil
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
F
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 op
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.
http://gerrit.c
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 s
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 sche
40 matches
Mail list logo
