Yep We can review the details directly on the patch. Additionally: I think a separate issue will be created, like "Documentation of the new API on the website"
Enrico 2017-09-12 10:07 GMT+02:00 Sijie Guo <guosi...@gmail.com>: > cool. just remember to put the new interfaces under > org.apache.bookkeeper.client.api. so we can separate client and server > module in future. > > - Sijie > > On Tue, Sep 12, 2017 at 1:04 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > OK, as there is not -1 I am marking this proposal as "Accepted". > > > > Thank you Jia and Sijie for voting and for comments from other guys at > the > > meeting. > > > > I wlil create and issue and send a pull request soon. > > It will take some time as we need tons of JavaDocs, we are going to > > introduce many *public *new classes, interfaces and method this time > > > > -- Enrico > > > > > > > > > > 2017-09-11 18:57 GMT+02:00 Sijie Guo <guosi...@gmail.com>: > > > > > Enrico, > > > > > > Feel free to close the vote if there is no -1. BP approval is a lazy > > > approval with no -1. > > > > > > - Sijie > > > > > > On Mon, Sep 11, 2017 at 2:27 AM, Enrico Olivelli <eolive...@gmail.com> > > > wrote: > > > > > > > Ping > > > > > > > > 2017-09-08 6:28 GMT+02:00 Jia Zhai <zhaiji...@gmail.com>: > > > > > > > > > +1 for the new design. > > > > > > > > > > On Thu, Sep 7, 2017 at 3:49 AM, Enrico Olivelli < > eolive...@gmail.com > > > > > > > > wrote: > > > > > > > > > > > Hi all, > > > > > > I would like to call a vote for this BookKeeper proposal > > > > > > > > > > > > This is the wiki page > > > > > > https://cwiki.apache.org/confluence/display/BOOKKEEPER/ > > > > > > BP-15+New+CreateLedger+API > > > > > > > > > > > > It is a new Client API for creating/opening ledgers. > > > > > > We are going to have separate interfaces for Writers and Readers > > and > > > we > > > > > are > > > > > > going to have a new fluent buider-style API for creating and > > opening > > > > > > ledgers. > > > > > > > > > > > > No real changes in semantics or in protocols to BookKeeper we are > > > only > > > > > > introducing a new modern and extensible Client API > > > > > > > > > > > > > > > > > > Regards > > > > > > Enrico Olivelli > > > > > > > > > > > > > > > > > > > > >
