You are viewing a plain text version of this content. The canonical link for it is here.
Posted to distributedlog-dev@bookkeeper.apache.org by Sijie Guo <gu...@gmail.com> on 2018/01/19 21:17:58 UTC

BP-26: Move distributedlog library into bookkeeper

Hi all,

I started a BP for proposing merging distributedlog library as part of
bookkeeper. So

1) bookkeeper has a higher level log api, provideing named logs over
numbered ledgers. it would make the usage of bookkeeper simpler and help
grow more usage in bookkeeper

2) it would make distributedlog library shipped as part of bookkeeper
release. It will make the users of distributedlog library easier, because
they just need install bookkeeper and use dlog api. An integrated release
procedure will provide better user experiences.

Details are proposed in the BP pull request:
https://github.com/apache/bookkeeper/pull/1025

Please take a look. Comments are welcome.

- Sijie

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
On Mon, Jan 29, 2018 at 12:02 PM, Henry Saputra <he...@gmail.com>
wrote:

> Just want to clarify, so issue #1068 is used to track PR #1069?
>

#1068 is for purely code movement to the given location. Since it is a code
movement, it is a bit hard to send out a PR for review, instead I push a
branch `BP-26/distributedlog_merge` and document the steps used for code
movement.

#1069 is the change based on #1068 to make those relocated modules compile
and build in bookkeeper project.

- Sijie


>
> On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:
>
> > Updates:
> >
> > I merged the corresponding modules into a bk branch. details are
> described
> > in https://github.com/apache/bookkeeper/issues/1068
> >
> > A subsequent PR to integrate those modules with BK pom files. In this PR,
> > this modules will not be built unless `-Pdistributedlog` is provided.
> > https://github.com/apache/bookkeeper/pull/1069
> >
> > Please help review it.
> >
> > - Sijie
> >
> > On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> > > This BP is accepted. see details at : https://github.com/apache/
> > > bookkeeper/pull/1025#issuecomment-360871025
> > >
> > > I've updated the master ticket #1024
> > > <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> > sub
> > > tasks.
> > >
> > > - Sijie
> > >
> > > On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> > >
> > >>
> > >>
> > >> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> > >> enrico.olivelli@diennea.com> wrote:
> > >>
> > >>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> > >>>
> > >>> Hi all,
> > >>>
> > >>> I started a BP for proposing merging distributedlog library as part
> of
> > >>> bookkeeper. So
> > >>>
> > >>> 1) bookkeeper has a higher level log api, provideing named logs over
> > >>> numbered ledgers. it would make the usage of bookkeeper simpler and
> > help
> > >>> grow more usage in bookkeeper
> > >>>
> > >>> 2) it would make distributedlog library shipped as part of bookkeeper
> > >>> release. It will make the users of distributedlog library easier,
> > because
> > >>> they just need install bookkeeper and use dlog api. An integrated
> > release
> > >>> procedure will provide better user experiences.
> > >>>
> > >>> Details are proposed in the BP pull request:
> > >>> https://github.com/apache/bookkeeper/pull/1025
> > >>>
> > >>> Please take a look. Comments are welcome.
> > >>>
> > >>>
> > >>> (I have already voted a big +1 on GitHub)
> > >>>
> > >>>
> > >>> Side question:
> > >>>
> > >>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> > we
> > >>> start an email thread ?
> > >>>
> > >>
> > >>
> > >> Yes +1 / approve on the pull request is the vote for the new BP
> > workflow.
> > >>
> > >>
> > >>>
> > >>> This is an important change, to me rules are not very clear with the
> > new
> > >>> github BP workflow, I knew on ASF it is important that all votes are
> > done
> > >>> by means of official mailing lists
> > >>>
> > >>
> > >> ASF requires all the conversations recorded in the ASF mailing list,
> > >> which all the github notifications are already sent and kept in
> > issues@b.a.o
> > >> mailing list.
> > >>
> > >>
> > >>>
> > >>> Enrico
> > >>>
> > >>>
> > >>>
> > >>>
> > >>>
> > >>>
> > >>>
> > >>> - Sijie
> > >>>
> > >>>
> > >>> --
> > >>>
> > >>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39)
> 0546
> > >>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> > >>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> > >>> Marketing Solutions http://www.diennea.com
> > >>>
> > >>> ________________________________
> > >>>
> > >>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital
> ed
> > >>> email marketing! http://www.magnews.it/newsletter/
> > >>>
> > >>> The information in this email is confidential and may be legally
> > >>> privileged. If you are not the intended recipient please notify the
> > sender
> > >>> immediately and destroy this email. Any unauthorized, direct or
> > indirect,
> > >>> disclosure, copying, storage, distribution or other use is strictly
> > >>> forbidden.
> > >>>
> > >>
> > >>
> > >
> >
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Henry Saputra <he...@gmail.com>.
Just want to clarify, so issue #1068 is used to track PR #1069?

On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:

> Updates:
>
> I merged the corresponding modules into a bk branch. details are described
> in https://github.com/apache/bookkeeper/issues/1068
>
> A subsequent PR to integrate those modules with BK pom files. In this PR,
> this modules will not be built unless `-Pdistributedlog` is provided.
> https://github.com/apache/bookkeeper/pull/1069
>
> Please help review it.
>
> - Sijie
>
> On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
>
> > This BP is accepted. see details at : https://github.com/apache/
> > bookkeeper/pull/1025#issuecomment-360871025
> >
> > I've updated the master ticket #1024
> > <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> sub
> > tasks.
> >
> > - Sijie
> >
> > On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> >>
> >>
> >> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> >> enrico.olivelli@diennea.com> wrote:
> >>
> >>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> >>>
> >>> Hi all,
> >>>
> >>> I started a BP for proposing merging distributedlog library as part of
> >>> bookkeeper. So
> >>>
> >>> 1) bookkeeper has a higher level log api, provideing named logs over
> >>> numbered ledgers. it would make the usage of bookkeeper simpler and
> help
> >>> grow more usage in bookkeeper
> >>>
> >>> 2) it would make distributedlog library shipped as part of bookkeeper
> >>> release. It will make the users of distributedlog library easier,
> because
> >>> they just need install bookkeeper and use dlog api. An integrated
> release
> >>> procedure will provide better user experiences.
> >>>
> >>> Details are proposed in the BP pull request:
> >>> https://github.com/apache/bookkeeper/pull/1025
> >>>
> >>> Please take a look. Comments are welcome.
> >>>
> >>>
> >>> (I have already voted a big +1 on GitHub)
> >>>
> >>>
> >>> Side question:
> >>>
> >>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> we
> >>> start an email thread ?
> >>>
> >>
> >>
> >> Yes +1 / approve on the pull request is the vote for the new BP
> workflow.
> >>
> >>
> >>>
> >>> This is an important change, to me rules are not very clear with the
> new
> >>> github BP workflow, I knew on ASF it is important that all votes are
> done
> >>> by means of official mailing lists
> >>>
> >>
> >> ASF requires all the conversations recorded in the ASF mailing list,
> >> which all the github notifications are already sent and kept in
> issues@b.a.o
> >> mailing list.
> >>
> >>
> >>>
> >>> Enrico
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>> - Sijie
> >>>
> >>>
> >>> --
> >>>
> >>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> >>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> >>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> >>> Marketing Solutions http://www.diennea.com
> >>>
> >>> ________________________________
> >>>
> >>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> >>> email marketing! http://www.magnews.it/newsletter/
> >>>
> >>> The information in this email is confidential and may be legally
> >>> privileged. If you are not the intended recipient please notify the
> sender
> >>> immediately and destroy this email. Any unauthorized, direct or
> indirect,
> >>> disclosure, copying, storage, distribution or other use is strictly
> >>> forbidden.
> >>>
> >>
> >>
> >
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Enrico Olivelli <eo...@gmail.com>.
That's a great work
I will review carefully

Thanks
Enrico

Il lun 29 gen 2018, 07:00 Sijie Guo <gu...@gmail.com> ha scritto:

> Updates:
>
> I merged the corresponding modules into a bk branch. details are described
> in https://github.com/apache/bookkeeper/issues/1068
>
> A subsequent PR to integrate those modules with BK pom files. In this PR,
> this modules will not be built unless `-Pdistributedlog` is provided.
> https://github.com/apache/bookkeeper/pull/1069
>
> Please help review it.
>
> - Sijie
>
> On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
>
> > This BP is accepted. see details at : https://github.com/apache/
> > bookkeeper/pull/1025#issuecomment-360871025
> >
> > I've updated the master ticket #1024
> > <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> sub
> > tasks.
> >
> > - Sijie
> >
> > On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> >>
> >>
> >> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> >> enrico.olivelli@diennea.com> wrote:
> >>
> >>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> >>>
> >>> Hi all,
> >>>
> >>> I started a BP for proposing merging distributedlog library as part of
> >>> bookkeeper. So
> >>>
> >>> 1) bookkeeper has a higher level log api, provideing named logs over
> >>> numbered ledgers. it would make the usage of bookkeeper simpler and
> help
> >>> grow more usage in bookkeeper
> >>>
> >>> 2) it would make distributedlog library shipped as part of bookkeeper
> >>> release. It will make the users of distributedlog library easier,
> because
> >>> they just need install bookkeeper and use dlog api. An integrated
> release
> >>> procedure will provide better user experiences.
> >>>
> >>> Details are proposed in the BP pull request:
> >>> https://github.com/apache/bookkeeper/pull/1025
> >>>
> >>> Please take a look. Comments are welcome.
> >>>
> >>>
> >>> (I have already voted a big +1 on GitHub)
> >>>
> >>>
> >>> Side question:
> >>>
> >>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> we
> >>> start an email thread ?
> >>>
> >>
> >>
> >> Yes +1 / approve on the pull request is the vote for the new BP
> workflow.
> >>
> >>
> >>>
> >>> This is an important change, to me rules are not very clear with the
> new
> >>> github BP workflow, I knew on ASF it is important that all votes are
> done
> >>> by means of official mailing lists
> >>>
> >>
> >> ASF requires all the conversations recorded in the ASF mailing list,
> >> which all the github notifications are already sent and kept in
> issues@b.a.o
> >> mailing list.
> >>
> >>
> >>>
> >>> Enrico
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>> - Sijie
> >>>
> >>>
> >>> --
> >>>
> >>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> >>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> >>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> >>> Marketing Solutions http://www.diennea.com
> >>>
> >>> ________________________________
> >>>
> >>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> >>> email marketing! http://www.magnews.it/newsletter/
> >>>
> >>> The information in this email is confidential and may be legally
> >>> privileged. If you are not the intended recipient please notify the
> sender
> >>> immediately and destroy this email. Any unauthorized, direct or
> indirect,
> >>> disclosure, copying, storage, distribution or other use is strictly
> >>> forbidden.
> >>>
> >>
> >>
> >
>


-- 


-- Enrico Olivelli

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Enrico Olivelli <eo...@gmail.com>.
Great ! Welcome DL to BK repo!

Enrico

Il mer 31 gen 2018, 22:38 Sijie Guo <gu...@gmail.com> ha scritto:

> Updates:
>
> I merged the PRs. so all the dlog modules are moved to bookkeeper repo and
> they are only activated when "-Ddistributedlog" is provided.
>
> Will work on updating CI and integrate the documentation.
>
> - Sijie
>
> On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:
>
> > Updates:
> >
> > I merged the corresponding modules into a bk branch. details are
> described
> > in https://github.com/apache/bookkeeper/issues/1068
> >
> > A subsequent PR to integrate those modules with BK pom files. In this PR,
> > this modules will not be built unless `-Pdistributedlog` is provided.
> > https://github.com/apache/bookkeeper/pull/1069
> >
> > Please help review it.
> >
> > - Sijie
> >
> > On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> >> This BP is accepted. see details at : https://github.com/apache/bo
> >> okkeeper/pull/1025#issuecomment-360871025
> >>
> >> I've updated the master ticket #1024
> >> <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> >> sub tasks.
> >>
> >> - Sijie
> >>
> >> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> >>
> >>>
> >>>
> >>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> >>> enrico.olivelli@diennea.com> wrote:
> >>>
> >>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> >>>>
> >>>> Hi all,
> >>>>
> >>>> I started a BP for proposing merging distributedlog library as part of
> >>>> bookkeeper. So
> >>>>
> >>>> 1) bookkeeper has a higher level log api, provideing named logs over
> >>>> numbered ledgers. it would make the usage of bookkeeper simpler and
> help
> >>>> grow more usage in bookkeeper
> >>>>
> >>>> 2) it would make distributedlog library shipped as part of bookkeeper
> >>>> release. It will make the users of distributedlog library easier,
> >>>> because
> >>>> they just need install bookkeeper and use dlog api. An integrated
> >>>> release
> >>>> procedure will provide better user experiences.
> >>>>
> >>>> Details are proposed in the BP pull request:
> >>>> https://github.com/apache/bookkeeper/pull/1025
> >>>>
> >>>> Please take a look. Comments are welcome.
> >>>>
> >>>>
> >>>> (I have already voted a big +1 on GitHub)
> >>>>
> >>>>
> >>>> Side question:
> >>>>
> >>>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> >>>> we start an email thread ?
> >>>>
> >>>
> >>>
> >>> Yes +1 / approve on the pull request is the vote for the new BP
> >>> workflow.
> >>>
> >>>
> >>>>
> >>>> This is an important change, to me rules are not very clear with the
> >>>> new github BP workflow, I knew on ASF it is important that all votes
> are
> >>>> done by means of official mailing lists
> >>>>
> >>>
> >>> ASF requires all the conversations recorded in the ASF mailing list,
> >>> which all the github notifications are already sent and kept in
> issues@b.a.o
> >>> mailing list.
> >>>
> >>>
> >>>>
> >>>> Enrico
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>> - Sijie
> >>>>
> >>>>
> >>>> --
> >>>>
> >>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> >>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> >>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> >>>> Marketing Solutions http://www.diennea.com
> >>>>
> >>>> ________________________________
> >>>>
> >>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> >>>> email marketing! http://www.magnews.it/newsletter/
> >>>>
> >>>> The information in this email is confidential and may be legally
> >>>> privileged. If you are not the intended recipient please notify the
> sender
> >>>> immediately and destroy this email. Any unauthorized, direct or
> indirect,
> >>>> disclosure, copying, storage, distribution or other use is strictly
> >>>> forbidden.
> >>>>
> >>>
> >>>
> >>
> >
>


-- 


-- Enrico Olivelli

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Enrico Olivelli <eo...@gmail.com>.
Great ! Welcome DL to BK repo!

Enrico

Il mer 31 gen 2018, 22:38 Sijie Guo <gu...@gmail.com> ha scritto:

> Updates:
>
> I merged the PRs. so all the dlog modules are moved to bookkeeper repo and
> they are only activated when "-Ddistributedlog" is provided.
>
> Will work on updating CI and integrate the documentation.
>
> - Sijie
>
> On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:
>
> > Updates:
> >
> > I merged the corresponding modules into a bk branch. details are
> described
> > in https://github.com/apache/bookkeeper/issues/1068
> >
> > A subsequent PR to integrate those modules with BK pom files. In this PR,
> > this modules will not be built unless `-Pdistributedlog` is provided.
> > https://github.com/apache/bookkeeper/pull/1069
> >
> > Please help review it.
> >
> > - Sijie
> >
> > On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> >> This BP is accepted. see details at : https://github.com/apache/bo
> >> okkeeper/pull/1025#issuecomment-360871025
> >>
> >> I've updated the master ticket #1024
> >> <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> >> sub tasks.
> >>
> >> - Sijie
> >>
> >> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> >>
> >>>
> >>>
> >>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> >>> enrico.olivelli@diennea.com> wrote:
> >>>
> >>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> >>>>
> >>>> Hi all,
> >>>>
> >>>> I started a BP for proposing merging distributedlog library as part of
> >>>> bookkeeper. So
> >>>>
> >>>> 1) bookkeeper has a higher level log api, provideing named logs over
> >>>> numbered ledgers. it would make the usage of bookkeeper simpler and
> help
> >>>> grow more usage in bookkeeper
> >>>>
> >>>> 2) it would make distributedlog library shipped as part of bookkeeper
> >>>> release. It will make the users of distributedlog library easier,
> >>>> because
> >>>> they just need install bookkeeper and use dlog api. An integrated
> >>>> release
> >>>> procedure will provide better user experiences.
> >>>>
> >>>> Details are proposed in the BP pull request:
> >>>> https://github.com/apache/bookkeeper/pull/1025
> >>>>
> >>>> Please take a look. Comments are welcome.
> >>>>
> >>>>
> >>>> (I have already voted a big +1 on GitHub)
> >>>>
> >>>>
> >>>> Side question:
> >>>>
> >>>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> >>>> we start an email thread ?
> >>>>
> >>>
> >>>
> >>> Yes +1 / approve on the pull request is the vote for the new BP
> >>> workflow.
> >>>
> >>>
> >>>>
> >>>> This is an important change, to me rules are not very clear with the
> >>>> new github BP workflow, I knew on ASF it is important that all votes
> are
> >>>> done by means of official mailing lists
> >>>>
> >>>
> >>> ASF requires all the conversations recorded in the ASF mailing list,
> >>> which all the github notifications are already sent and kept in
> issues@b.a.o
> >>> mailing list.
> >>>
> >>>
> >>>>
> >>>> Enrico
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>>
> >>>> - Sijie
> >>>>
> >>>>
> >>>> --
> >>>>
> >>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> >>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> >>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> >>>> Marketing Solutions http://www.diennea.com
> >>>>
> >>>> ________________________________
> >>>>
> >>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> >>>> email marketing! http://www.magnews.it/newsletter/
> >>>>
> >>>> The information in this email is confidential and may be legally
> >>>> privileged. If you are not the intended recipient please notify the
> sender
> >>>> immediately and destroy this email. Any unauthorized, direct or
> indirect,
> >>>> disclosure, copying, storage, distribution or other use is strictly
> >>>> forbidden.
> >>>>
> >>>
> >>>
> >>
> >
>


-- 


-- Enrico Olivelli

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
Updates:

I merged the PRs. so all the dlog modules are moved to bookkeeper repo and
they are only activated when "-Ddistributedlog" is provided.

Will work on updating CI and integrate the documentation.

- Sijie

On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:

> Updates:
>
> I merged the corresponding modules into a bk branch. details are described
> in https://github.com/apache/bookkeeper/issues/1068
>
> A subsequent PR to integrate those modules with BK pom files. In this PR,
> this modules will not be built unless `-Pdistributedlog` is provided.
> https://github.com/apache/bookkeeper/pull/1069
>
> Please help review it.
>
> - Sijie
>
> On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
>
>> This BP is accepted. see details at : https://github.com/apache/bo
>> okkeeper/pull/1025#issuecomment-360871025
>>
>> I've updated the master ticket #1024
>> <https://github.com/apache/bookkeeper/issues/1024> with corresponding
>> sub tasks.
>>
>> - Sijie
>>
>> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
>>
>>>
>>>
>>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
>>> enrico.olivelli@diennea.com> wrote:
>>>
>>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>>>
>>>> Hi all,
>>>>
>>>> I started a BP for proposing merging distributedlog library as part of
>>>> bookkeeper. So
>>>>
>>>> 1) bookkeeper has a higher level log api, provideing named logs over
>>>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>>>> grow more usage in bookkeeper
>>>>
>>>> 2) it would make distributedlog library shipped as part of bookkeeper
>>>> release. It will make the users of distributedlog library easier,
>>>> because
>>>> they just need install bookkeeper and use dlog api. An integrated
>>>> release
>>>> procedure will provide better user experiences.
>>>>
>>>> Details are proposed in the BP pull request:
>>>> https://github.com/apache/bookkeeper/pull/1025
>>>>
>>>> Please take a look. Comments are welcome.
>>>>
>>>>
>>>> (I have already voted a big +1 on GitHub)
>>>>
>>>>
>>>> Side question:
>>>>
>>>> Will the +1 on the Pull Request be valid as official "VOTE" or should
>>>> we start an email thread ?
>>>>
>>>
>>>
>>> Yes +1 / approve on the pull request is the vote for the new BP
>>> workflow.
>>>
>>>
>>>>
>>>> This is an important change, to me rules are not very clear with the
>>>> new github BP workflow, I knew on ASF it is important that all votes are
>>>> done by means of official mailing lists
>>>>
>>>
>>> ASF requires all the conversations recorded in the ASF mailing list,
>>> which all the github notifications are already sent and kept in issues@b.a.o
>>> mailing list.
>>>
>>>
>>>>
>>>> Enrico
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> - Sijie
>>>>
>>>>
>>>> --
>>>>
>>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>>>> Marketing Solutions http://www.diennea.com
>>>>
>>>> ________________________________
>>>>
>>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>>>> email marketing! http://www.magnews.it/newsletter/
>>>>
>>>> The information in this email is confidential and may be legally
>>>> privileged. If you are not the intended recipient please notify the sender
>>>> immediately and destroy this email. Any unauthorized, direct or indirect,
>>>> disclosure, copying, storage, distribution or other use is strictly
>>>> forbidden.
>>>>
>>>
>>>
>>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Enrico Olivelli <eo...@gmail.com>.
That's a great work
I will review carefully

Thanks
Enrico

Il lun 29 gen 2018, 07:00 Sijie Guo <gu...@gmail.com> ha scritto:

> Updates:
>
> I merged the corresponding modules into a bk branch. details are described
> in https://github.com/apache/bookkeeper/issues/1068
>
> A subsequent PR to integrate those modules with BK pom files. In this PR,
> this modules will not be built unless `-Pdistributedlog` is provided.
> https://github.com/apache/bookkeeper/pull/1069
>
> Please help review it.
>
> - Sijie
>
> On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
>
> > This BP is accepted. see details at : https://github.com/apache/
> > bookkeeper/pull/1025#issuecomment-360871025
> >
> > I've updated the master ticket #1024
> > <https://github.com/apache/bookkeeper/issues/1024> with corresponding
> sub
> > tasks.
> >
> > - Sijie
> >
> > On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
> >
> >>
> >>
> >> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> >> enrico.olivelli@diennea.com> wrote:
> >>
> >>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
> >>>
> >>> Hi all,
> >>>
> >>> I started a BP for proposing merging distributedlog library as part of
> >>> bookkeeper. So
> >>>
> >>> 1) bookkeeper has a higher level log api, provideing named logs over
> >>> numbered ledgers. it would make the usage of bookkeeper simpler and
> help
> >>> grow more usage in bookkeeper
> >>>
> >>> 2) it would make distributedlog library shipped as part of bookkeeper
> >>> release. It will make the users of distributedlog library easier,
> because
> >>> they just need install bookkeeper and use dlog api. An integrated
> release
> >>> procedure will provide better user experiences.
> >>>
> >>> Details are proposed in the BP pull request:
> >>> https://github.com/apache/bookkeeper/pull/1025
> >>>
> >>> Please take a look. Comments are welcome.
> >>>
> >>>
> >>> (I have already voted a big +1 on GitHub)
> >>>
> >>>
> >>> Side question:
> >>>
> >>> Will the +1 on the Pull Request be valid as official "VOTE" or should
> we
> >>> start an email thread ?
> >>>
> >>
> >>
> >> Yes +1 / approve on the pull request is the vote for the new BP
> workflow.
> >>
> >>
> >>>
> >>> This is an important change, to me rules are not very clear with the
> new
> >>> github BP workflow, I knew on ASF it is important that all votes are
> done
> >>> by means of official mailing lists
> >>>
> >>
> >> ASF requires all the conversations recorded in the ASF mailing list,
> >> which all the github notifications are already sent and kept in
> issues@b.a.o
> >> mailing list.
> >>
> >>
> >>>
> >>> Enrico
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>>
> >>> - Sijie
> >>>
> >>>
> >>> --
> >>>
> >>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> >>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> >>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> >>> Marketing Solutions http://www.diennea.com
> >>>
> >>> ________________________________
> >>>
> >>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> >>> email marketing! http://www.magnews.it/newsletter/
> >>>
> >>> The information in this email is confidential and may be legally
> >>> privileged. If you are not the intended recipient please notify the
> sender
> >>> immediately and destroy this email. Any unauthorized, direct or
> indirect,
> >>> disclosure, copying, storage, distribution or other use is strictly
> >>> forbidden.
> >>>
> >>
> >>
> >
>


-- 


-- Enrico Olivelli

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
Updates:

I merged the PRs. so all the dlog modules are moved to bookkeeper repo and
they are only activated when "-Ddistributedlog" is provided.

Will work on updating CI and integrate the documentation.

- Sijie

On Sun, Jan 28, 2018 at 9:59 PM, Sijie Guo <gu...@gmail.com> wrote:

> Updates:
>
> I merged the corresponding modules into a bk branch. details are described
> in https://github.com/apache/bookkeeper/issues/1068
>
> A subsequent PR to integrate those modules with BK pom files. In this PR,
> this modules will not be built unless `-Pdistributedlog` is provided.
> https://github.com/apache/bookkeeper/pull/1069
>
> Please help review it.
>
> - Sijie
>
> On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:
>
>> This BP is accepted. see details at : https://github.com/apache/bo
>> okkeeper/pull/1025#issuecomment-360871025
>>
>> I've updated the master ticket #1024
>> <https://github.com/apache/bookkeeper/issues/1024> with corresponding
>> sub tasks.
>>
>> - Sijie
>>
>> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
>>
>>>
>>>
>>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
>>> enrico.olivelli@diennea.com> wrote:
>>>
>>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>>>
>>>> Hi all,
>>>>
>>>> I started a BP for proposing merging distributedlog library as part of
>>>> bookkeeper. So
>>>>
>>>> 1) bookkeeper has a higher level log api, provideing named logs over
>>>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>>>> grow more usage in bookkeeper
>>>>
>>>> 2) it would make distributedlog library shipped as part of bookkeeper
>>>> release. It will make the users of distributedlog library easier,
>>>> because
>>>> they just need install bookkeeper and use dlog api. An integrated
>>>> release
>>>> procedure will provide better user experiences.
>>>>
>>>> Details are proposed in the BP pull request:
>>>> https://github.com/apache/bookkeeper/pull/1025
>>>>
>>>> Please take a look. Comments are welcome.
>>>>
>>>>
>>>> (I have already voted a big +1 on GitHub)
>>>>
>>>>
>>>> Side question:
>>>>
>>>> Will the +1 on the Pull Request be valid as official "VOTE" or should
>>>> we start an email thread ?
>>>>
>>>
>>>
>>> Yes +1 / approve on the pull request is the vote for the new BP
>>> workflow.
>>>
>>>
>>>>
>>>> This is an important change, to me rules are not very clear with the
>>>> new github BP workflow, I knew on ASF it is important that all votes are
>>>> done by means of official mailing lists
>>>>
>>>
>>> ASF requires all the conversations recorded in the ASF mailing list,
>>> which all the github notifications are already sent and kept in issues@b.a.o
>>> mailing list.
>>>
>>>
>>>>
>>>> Enrico
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> - Sijie
>>>>
>>>>
>>>> --
>>>>
>>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>>>> Marketing Solutions http://www.diennea.com
>>>>
>>>> ________________________________
>>>>
>>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>>>> email marketing! http://www.magnews.it/newsletter/
>>>>
>>>> The information in this email is confidential and may be legally
>>>> privileged. If you are not the intended recipient please notify the sender
>>>> immediately and destroy this email. Any unauthorized, direct or indirect,
>>>> disclosure, copying, storage, distribution or other use is strictly
>>>> forbidden.
>>>>
>>>
>>>
>>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
Updates:

I merged the corresponding modules into a bk branch. details are described
in https://github.com/apache/bookkeeper/issues/1068

A subsequent PR to integrate those modules with BK pom files. In this PR,
this modules will not be built unless `-Pdistributedlog` is provided.
https://github.com/apache/bookkeeper/pull/1069

Please help review it.

- Sijie

On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:

> This BP is accepted. see details at : https://github.com/apache/
> bookkeeper/pull/1025#issuecomment-360871025
>
> I've updated the master ticket #1024
> <https://github.com/apache/bookkeeper/issues/1024> with corresponding sub
> tasks.
>
> - Sijie
>
> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
>
>>
>>
>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
>> enrico.olivelli@diennea.com> wrote:
>>
>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>>
>>> Hi all,
>>>
>>> I started a BP for proposing merging distributedlog library as part of
>>> bookkeeper. So
>>>
>>> 1) bookkeeper has a higher level log api, provideing named logs over
>>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>>> grow more usage in bookkeeper
>>>
>>> 2) it would make distributedlog library shipped as part of bookkeeper
>>> release. It will make the users of distributedlog library easier, because
>>> they just need install bookkeeper and use dlog api. An integrated release
>>> procedure will provide better user experiences.
>>>
>>> Details are proposed in the BP pull request:
>>> https://github.com/apache/bookkeeper/pull/1025
>>>
>>> Please take a look. Comments are welcome.
>>>
>>>
>>> (I have already voted a big +1 on GitHub)
>>>
>>>
>>> Side question:
>>>
>>> Will the +1 on the Pull Request be valid as official "VOTE" or should we
>>> start an email thread ?
>>>
>>
>>
>> Yes +1 / approve on the pull request is the vote for the new BP workflow.
>>
>>
>>>
>>> This is an important change, to me rules are not very clear with the new
>>> github BP workflow, I knew on ASF it is important that all votes are done
>>> by means of official mailing lists
>>>
>>
>> ASF requires all the conversations recorded in the ASF mailing list,
>> which all the github notifications are already sent and kept in issues@b.a.o
>> mailing list.
>>
>>
>>>
>>> Enrico
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> - Sijie
>>>
>>>
>>> --
>>>
>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>>> Marketing Solutions http://www.diennea.com
>>>
>>> ________________________________
>>>
>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>>> email marketing! http://www.magnews.it/newsletter/
>>>
>>> The information in this email is confidential and may be legally
>>> privileged. If you are not the intended recipient please notify the sender
>>> immediately and destroy this email. Any unauthorized, direct or indirect,
>>> disclosure, copying, storage, distribution or other use is strictly
>>> forbidden.
>>>
>>
>>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
Updates:

I merged the corresponding modules into a bk branch. details are described
in https://github.com/apache/bookkeeper/issues/1068

A subsequent PR to integrate those modules with BK pom files. In this PR,
this modules will not be built unless `-Pdistributedlog` is provided.
https://github.com/apache/bookkeeper/pull/1069

Please help review it.

- Sijie

On Fri, Jan 26, 2018 at 11:00 AM, Sijie Guo <gu...@gmail.com> wrote:

> This BP is accepted. see details at : https://github.com/apache/
> bookkeeper/pull/1025#issuecomment-360871025
>
> I've updated the master ticket #1024
> <https://github.com/apache/bookkeeper/issues/1024> with corresponding sub
> tasks.
>
> - Sijie
>
> On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:
>
>>
>>
>> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
>> enrico.olivelli@diennea.com> wrote:
>>
>>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>>
>>> Hi all,
>>>
>>> I started a BP for proposing merging distributedlog library as part of
>>> bookkeeper. So
>>>
>>> 1) bookkeeper has a higher level log api, provideing named logs over
>>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>>> grow more usage in bookkeeper
>>>
>>> 2) it would make distributedlog library shipped as part of bookkeeper
>>> release. It will make the users of distributedlog library easier, because
>>> they just need install bookkeeper and use dlog api. An integrated release
>>> procedure will provide better user experiences.
>>>
>>> Details are proposed in the BP pull request:
>>> https://github.com/apache/bookkeeper/pull/1025
>>>
>>> Please take a look. Comments are welcome.
>>>
>>>
>>> (I have already voted a big +1 on GitHub)
>>>
>>>
>>> Side question:
>>>
>>> Will the +1 on the Pull Request be valid as official "VOTE" or should we
>>> start an email thread ?
>>>
>>
>>
>> Yes +1 / approve on the pull request is the vote for the new BP workflow.
>>
>>
>>>
>>> This is an important change, to me rules are not very clear with the new
>>> github BP workflow, I knew on ASF it is important that all votes are done
>>> by means of official mailing lists
>>>
>>
>> ASF requires all the conversations recorded in the ASF mailing list,
>> which all the github notifications are already sent and kept in issues@b.a.o
>> mailing list.
>>
>>
>>>
>>> Enrico
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> - Sijie
>>>
>>>
>>> --
>>>
>>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>>> Marketing Solutions http://www.diennea.com
>>>
>>> ________________________________
>>>
>>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>>> email marketing! http://www.magnews.it/newsletter/
>>>
>>> The information in this email is confidential and may be legally
>>> privileged. If you are not the intended recipient please notify the sender
>>> immediately and destroy this email. Any unauthorized, direct or indirect,
>>> disclosure, copying, storage, distribution or other use is strictly
>>> forbidden.
>>>
>>
>>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
This BP is accepted. see details at :
https://github.com/apache/bookkeeper/pull/1025#issuecomment-360871025

I've updated the master ticket #1024
<https://github.com/apache/bookkeeper/issues/1024> with corresponding sub
tasks.

- Sijie

On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:

>
>
> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> enrico.olivelli@diennea.com> wrote:
>
>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>
>> Hi all,
>>
>> I started a BP for proposing merging distributedlog library as part of
>> bookkeeper. So
>>
>> 1) bookkeeper has a higher level log api, provideing named logs over
>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>> grow more usage in bookkeeper
>>
>> 2) it would make distributedlog library shipped as part of bookkeeper
>> release. It will make the users of distributedlog library easier, because
>> they just need install bookkeeper and use dlog api. An integrated release
>> procedure will provide better user experiences.
>>
>> Details are proposed in the BP pull request:
>> https://github.com/apache/bookkeeper/pull/1025
>>
>> Please take a look. Comments are welcome.
>>
>>
>> (I have already voted a big +1 on GitHub)
>>
>>
>> Side question:
>>
>> Will the +1 on the Pull Request be valid as official "VOTE" or should we
>> start an email thread ?
>>
>
>
> Yes +1 / approve on the pull request is the vote for the new BP workflow.
>
>
>>
>> This is an important change, to me rules are not very clear with the new
>> github BP workflow, I knew on ASF it is important that all votes are done
>> by means of official mailing lists
>>
>
> ASF requires all the conversations recorded in the ASF mailing list, which
> all the github notifications are already sent and kept in issues@b.a.o
> mailing list.
>
>
>>
>> Enrico
>>
>>
>>
>>
>>
>>
>>
>> - Sijie
>>
>>
>> --
>>
>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>> Marketing Solutions http://www.diennea.com
>>
>> ________________________________
>>
>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>> email marketing! http://www.magnews.it/newsletter/
>>
>> The information in this email is confidential and may be legally
>> privileged. If you are not the intended recipient please notify the sender
>> immediately and destroy this email. Any unauthorized, direct or indirect,
>> disclosure, copying, storage, distribution or other use is strictly
>> forbidden.
>>
>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
This BP is accepted. see details at :
https://github.com/apache/bookkeeper/pull/1025#issuecomment-360871025

I've updated the master ticket #1024
<https://github.com/apache/bookkeeper/issues/1024> with corresponding sub
tasks.

- Sijie

On Mon, Jan 22, 2018 at 6:11 AM, Sijie Guo <gu...@gmail.com> wrote:

>
>
> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> enrico.olivelli@diennea.com> wrote:
>
>> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>>
>> Hi all,
>>
>> I started a BP for proposing merging distributedlog library as part of
>> bookkeeper. So
>>
>> 1) bookkeeper has a higher level log api, provideing named logs over
>> numbered ledgers. it would make the usage of bookkeeper simpler and help
>> grow more usage in bookkeeper
>>
>> 2) it would make distributedlog library shipped as part of bookkeeper
>> release. It will make the users of distributedlog library easier, because
>> they just need install bookkeeper and use dlog api. An integrated release
>> procedure will provide better user experiences.
>>
>> Details are proposed in the BP pull request:
>> https://github.com/apache/bookkeeper/pull/1025
>>
>> Please take a look. Comments are welcome.
>>
>>
>> (I have already voted a big +1 on GitHub)
>>
>>
>> Side question:
>>
>> Will the +1 on the Pull Request be valid as official "VOTE" or should we
>> start an email thread ?
>>
>
>
> Yes +1 / approve on the pull request is the vote for the new BP workflow.
>
>
>>
>> This is an important change, to me rules are not very clear with the new
>> github BP workflow, I knew on ASF it is important that all votes are done
>> by means of official mailing lists
>>
>
> ASF requires all the conversations recorded in the ASF mailing list, which
> all the github notifications are already sent and kept in issues@b.a.o
> mailing list.
>
>
>>
>> Enrico
>>
>>
>>
>>
>>
>>
>>
>> - Sijie
>>
>>
>> --
>>
>> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
>> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
>> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
>> Marketing Solutions http://www.diennea.com
>>
>> ________________________________
>>
>> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
>> email marketing! http://www.magnews.it/newsletter/
>>
>> The information in this email is confidential and may be legally
>> privileged. If you are not the intended recipient please notify the sender
>> immediately and destroy this email. Any unauthorized, direct or indirect,
>> disclosure, copying, storage, distribution or other use is strictly
>> forbidden.
>>
>
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Sijie Guo <gu...@gmail.com>.
On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
enrico.olivelli@diennea.com> wrote:

> Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:
>
> Hi all,
>
> I started a BP for proposing merging distributedlog library as part of
> bookkeeper. So
>
> 1) bookkeeper has a higher level log api, provideing named logs over
> numbered ledgers. it would make the usage of bookkeeper simpler and help
> grow more usage in bookkeeper
>
> 2) it would make distributedlog library shipped as part of bookkeeper
> release. It will make the users of distributedlog library easier, because
> they just need install bookkeeper and use dlog api. An integrated release
> procedure will provide better user experiences.
>
> Details are proposed in the BP pull request:
> https://github.com/apache/bookkeeper/pull/1025
>
> Please take a look. Comments are welcome.
>
>
> (I have already voted a big +1 on GitHub)
>
>
> Side question:
>
> Will the +1 on the Pull Request be valid as official "VOTE" or should we
> start an email thread ?
>


Yes +1 / approve on the pull request is the vote for the new BP workflow.


>
> This is an important change, to me rules are not very clear with the new
> github BP workflow, I knew on ASF it is important that all votes are done
> by means of official mailing lists
>

ASF requires all the conversations recorded in the ASF mailing list, which
all the github notifications are already sent and kept in issues@b.a.o
mailing list.


>
> Enrico
>
>
>
>
>
>
>
> - Sijie
>
>
> --
>
> Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546
> 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews -
> E-mail Marketing Solutions http://www.magnews.it Diennea - Digital
> Marketing Solutions http://www.diennea.com
>
> ________________________________
>
> Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed
> email marketing! http://www.magnews.it/newsletter/
>
> The information in this email is confidential and may be legally
> privileged. If you are not the intended recipient please notify the sender
> immediately and destroy this email. Any unauthorized, direct or indirect,
> disclosure, copying, storage, distribution or other use is strictly
> forbidden.
>

Re: BP-26: Move distributedlog library into bookkeeper

Posted by Enrico Olivelli - Diennea <en...@diennea.com>.
Il giorno ven, 19/01/2018 alle 13.17 -0800, Sijie Guo ha scritto:

Hi all,

I started a BP for proposing merging distributedlog library as part of
bookkeeper. So

1) bookkeeper has a higher level log api, provideing named logs over
numbered ledgers. it would make the usage of bookkeeper simpler and help
grow more usage in bookkeeper

2) it would make distributedlog library shipped as part of bookkeeper
release. It will make the users of distributedlog library easier, because
they just need install bookkeeper and use dlog api. An integrated release
procedure will provide better user experiences.

Details are proposed in the BP pull request:
https://github.com/apache/bookkeeper/pull/1025

Please take a look. Comments are welcome.


(I have already voted a big +1 on GitHub)


Side question:

Will the +1 on the Pull Request be valid as official "VOTE" or should we start an email thread ?

This is an important change, to me rules are not very clear with the new github BP workflow, I knew on ASF it is important that all votes are done by means of official mailing lists


Enrico







- Sijie


--

Enrico Olivelli Software Development Manager @Diennea Tel.: (+39) 0546 066100 - Int. 925 Viale G.Marconi 30/14 - 48018 Faenza (RA) MagNews - E-mail Marketing Solutions http://www.magnews.it Diennea - Digital Marketing Solutions http://www.diennea.com

________________________________

Iscriviti alla nostra newsletter per rimanere aggiornato su digital ed email marketing! http://www.magnews.it/newsletter/

The information in this email is confidential and may be legally privileged. If you are not the intended recipient please notify the sender immediately and destroy this email. Any unauthorized, direct or indirect, disclosure, copying, storage, distribution or other use is strictly forbidden.