You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bookkeeper.apache.org by Sijie Guo <si...@apache.org> on 2016/10/13 05:56:03 UTC
Re: BOOKKEEPER-612 RegionAwarePlacement Policy Progress
FYI. I just merged the pull request. So the region aware placement policy
is in now.
- Sijie
On Mon, Sep 26, 2016 at 7:28 AM, Sijie Guo <si...@apache.org> wrote:
> Enrico,
>
> Sorry. I am on vacation. Didn't have network to check the email until now.
>
> Matteo or JV can help merging the pull request too.
>
> - Sijie
>
> On Fri, Sep 23, 2016 at 5:04 PM, Enrico Olivelli <eo...@gmail.com>
> wrote:
>
>> Thank you Flavio,
>>
>> there is no hurry, I can merge it locally and work on a temporary
>> version, no problem
>>
>> -- Enrico
>>
>> 2016-09-23 10:00 GMT+02:00 Flavio Junqueira <fp...@apache.org>:
>> > Enrico,
>> >
>> > The PR for BK-612 has been reviewed, right? We need to wait for Sijie
>> to show up because he has been the main committer reviewing it, I'd rather
>> not merge without his +1. Perhaps he is busy or away, so let's wait.
>> >
>> > -Flavio
>> >
>> >> On 23 Sep 2016, at 08:06, Enrico Olivelli <eo...@gmail.com> wrote:
>> >>
>> >> (I'm re-sending this email from another account, as I think the prev
>> >> message went to spam folder)
>> >>
>> >> Hi BookKeepers,
>> >>
>> >> I would like to ping on BOOKKEEPER-612, which has a pending PR.
>> >>
>> >> It is actually blocking the work on
>> >> https://issues.apache.org/jira/browse/BOOKKEEPER-912 Allow
>> >> EnsemblePlacementPolicy to choose bookies using ledger custom data
>> >> (multitenancy support) as this is changes the signature of methods on
>> >> EnsemblePlacementPolicy.
>> >>
>> >> Does anyone have time to review so that it can be committed ?
>> >>
>> >> For me BOOKKEEEPER-912 is a very important feature and I would like to
>> >> work on it for 4.5.0
>> >>
>> >> This is the issue
>> >> https://issues.apache.org/jira/browse/BOOKKEEPER-612
>> >>
>> >> This is the PR from Robin
>> >> https://github.com/apache/bookkeeper/pull/56
>> >>
>> >>
>> >> Thank you very much
>> >>
>> >> Enrico
>> >
>>
>
>
Re: BOOKKEEPER-612 RegionAwarePlacement Policy Progress
Posted by Enrico Olivelli <eo...@gmail.com>.
Thank you!
Il gio 13 ott 2016, 07:56 Sijie Guo <si...@apache.org> ha scritto:
> FYI. I just merged the pull request. So the region aware placement policy
> is in now.
>
> - Sijie
>
> On Mon, Sep 26, 2016 at 7:28 AM, Sijie Guo <si...@apache.org> wrote:
>
> > Enrico,
> >
> > Sorry. I am on vacation. Didn't have network to check the email until
> now.
> >
> > Matteo or JV can help merging the pull request too.
> >
> > - Sijie
> >
> > On Fri, Sep 23, 2016 at 5:04 PM, Enrico Olivelli <eo...@gmail.com>
> > wrote:
> >
> >> Thank you Flavio,
> >>
> >> there is no hurry, I can merge it locally and work on a temporary
> >> version, no problem
> >>
> >> -- Enrico
> >>
> >> 2016-09-23 10:00 GMT+02:00 Flavio Junqueira <fp...@apache.org>:
> >> > Enrico,
> >> >
> >> > The PR for BK-612 has been reviewed, right? We need to wait for Sijie
> >> to show up because he has been the main committer reviewing it, I'd
> rather
> >> not merge without his +1. Perhaps he is busy or away, so let's wait.
> >> >
> >> > -Flavio
> >> >
> >> >> On 23 Sep 2016, at 08:06, Enrico Olivelli <eo...@gmail.com>
> wrote:
> >> >>
> >> >> (I'm re-sending this email from another account, as I think the prev
> >> >> message went to spam folder)
> >> >>
> >> >> Hi BookKeepers,
> >> >>
> >> >> I would like to ping on BOOKKEEPER-612, which has a pending PR.
> >> >>
> >> >> It is actually blocking the work on
> >> >> https://issues.apache.org/jira/browse/BOOKKEEPER-912 Allow
> >> >> EnsemblePlacementPolicy to choose bookies using ledger custom data
> >> >> (multitenancy support) as this is changes the signature of methods on
> >> >> EnsemblePlacementPolicy.
> >> >>
> >> >> Does anyone have time to review so that it can be committed ?
> >> >>
> >> >> For me BOOKKEEEPER-912 is a very important feature and I would like
> to
> >> >> work on it for 4.5.0
> >> >>
> >> >> This is the issue
> >> >> https://issues.apache.org/jira/browse/BOOKKEEPER-612
> >> >>
> >> >> This is the PR from Robin
> >> >> https://github.com/apache/bookkeeper/pull/56
> >> >>
> >> >>
> >> >> Thank you very much
> >> >>
> >> >> Enrico
> >> >
> >>
> >
> >
>
--
-- Enrico Olivelli