On Mon, Jan 29, 2018 at 12:02 PM, Henry Saputra <henry.sapu...@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 <guosi...@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 <guosi...@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 <guosi...@gmail.com> wrote:
> > >
> > >>
> > >>
> > >> On Mon, Jan 22, 2018 at 3:44 AM, Enrico Olivelli - Diennea <
> > >> enrico.olive...@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.
> > >>>
> > >>
> > >>
> > >
> >
>

Reply via email to