Hi - This is great. Now that this is in the two projects (which share committers) will need to co-ordinate releases.
Keep in mind the next Pulsar release needs to be based on a particular BookKeeper release. An Apache project cannot release against another project's SNAPSHOT since that has not been validated by their release process. Best Regards, Dave > On Mar 2, 2018, at 3:23 AM, Ivan Kelly <iv...@apache.org> wrote: > > Great to see all these changes in! > > -Ivan > > On Thu, Mar 1, 2018 at 8:14 PM, Enrico Olivelli <eolive...@gmail.com> wrote: >> Great! >> Congrats >> >> Do you have any number to compare performances? >> >> Enrico >> >> Il gio 1 mar 2018, 19:06 Sijie Guo <guosi...@gmail.com> ha scritto: >> >>> Awesome work, Ivan & Jia! >>> >>> Congrats Pulsar community! >>> >>> - Sijie >>> >>> On Thu, Mar 1, 2018 at 9:53 AM, Matteo Merli <mme...@apache.org> wrote: >>> >>>> In Pulsar master branch, we have switched the BookKeeper dependency from >>>> the Yahoo fork to the 4.7.0-SNAPSHOT version of BookKeeper from main >>> master >>>> branch. >>>> >>>> All the the changes that were in the Yahoo fork, which was based on 4.3.1 >>>> release, have been already merged upstream. >>>> >>>> This was a big effort that took ~ 1 year to get through. There were 246 >>>> commits to merge into a codebase that slightly changed in a 4 years >>>> timespan. For the curious, this is the spreadsheet we used to track the >>>> merging. >>>> https://docs.google.com/spreadsheets/d/1jAy3EfjViqNEKpCKpWiRv- >>>> PCZGzdjwm_PclL7Obog4Q/ >>>> >>>> I would like to call out Ivan Kelly & Jia Zhai for giving a big push of >>>> porting many of the changes into BookKeeper and the BookKeeper community >>>> for being very receptive and helpful in getting this load of changes back >>>> into mainline. >>>> >>>> >>>> Matteo >>>> -- >>>> Matteo Merli >>>> <mme...@apache.org> >>>> >>> >> -- >> >> >> -- Enrico Olivelli
signature.asc
Description: Message signed with OpenPGP