Awesome! Thanks Charles! Jihoon
On Wed, Aug 8, 2018 at 1:16 PM Gian Merlino <g...@apache.org> wrote: > Thanks, it will be nice to see! > > On Wed, Aug 8, 2018 at 1:15 PM Charles Allen <charles.al...@snap.com > .invalid> > wrote: > > > I don't think it should be a blocker to release, but I have to run perf > > tests for rollouts anyways so I figured I'd publish what I find :-P > > > > Cheers, > > Charles Allen > > > > > > On Wed, Aug 8, 2018 at 12:33 PM Gian Merlino <g...@apache.org> wrote: > > > > > That being said, Charles I am definitely looking forward to your report > > of > > > what the upgrade from 0.11 -> 0.12.2-rc1 is like in your cluster! > > > > > > On Wed, Aug 8, 2018 at 12:30 PM Gian Merlino <g...@apache.org> wrote: > > > > > > > My thought is that recently we have started doing small bug-fix > > releases > > > > more often (0.12.1 and 0.12.2 were both small releases) and I think > it > > > > makes sense to continue this practice. It makes sense to get them out > > > > quickly, since shipping bug fixes is good. IMO trying to validate bug > > fix > > > > releases within the customary Apache style 72 hour voting period is a > > > good > > > > goal. > > > > > > > > On the other hand we do strive to put out high quality releases, and > we > > > > don't want bug fix releases to introduce regressions. Testing every > > > single > > > > patch in real clusters is an important part of that. All I can do is > > > > encourage people running real clusters to deploy RCs as fast as they > > can! > > > > Fwiw, we have already incorporated all the 0.12.2 patches into our > > Imply > > > > distro of Druid and already have a good number users running them. So > > my > > > +1 > > > > earlier incorporated knowledge that the patches have been validated > in > > > that > > > > way. > > > > > > > > I agree with Jihoon that we will probably end up doing an 0.12.3 > soon, > > to > > > > fix the issues he mentioned and a couple of others as well. > > > > > > > > On Wed, Aug 8, 2018 at 12:07 PM Jihoon Son <jihoon...@apache.org> > > wrote: > > > > > > > >> Charles, thank you for doing performance evaluation! Performance > > numbers > > > >> are always good and helpful. > > > >> > > > >> However, IMO, any kind of performance degradation shouldn't be a > > blocker > > > >> for this release. 0.12.2 is a minor release and contains only bug > > fixes. > > > >> https://github.com/apache/incubator-druid/pull/5878/files is the > only > > > one > > > >> tagged with 'Performance', but it can be regarded as a more like a > > code > > > >> bug > > > >> rather than architectural performance issue. > > > >> > > > >> Instead, those kinds of performance tests should be performed per > > major > > > >> release to catch any kinds of unexpected performance change. They > can > > > be a > > > >> blocker if we find any performance regression. > > > >> > > > >> Also, if you find any performance regression for this release, we > > > probably > > > >> make another minor release. I think some bug fixes (e.g., > > > >> https://github.com/apache/incubator-druid/issues/6124, > > > >> https://github.com/apache/incubator-druid/issues/6123) are also > worth > > > to > > > >> be > > > >> included in the minor release. > > > >> > > > >> What do you think? > > > >> > > > >> Best, > > > >> Jihoon > > > >> > > > >> On Wed, Aug 8, 2018 at 10:18 AM Charles Allen > > > >> <charles.al...@snap.com.invalid> wrote: > > > >> > > > >> > I'm hoping to have some numbers for any performance changes or > other > > > >> > impacts in the next few days (rollouts on big clusters take a long > > > >> time). I > > > >> > am neutral until the numbers come in. Preliminary indicators show > no > > > >> > significant regression since the 0.11.x series. More data is > > expected > > > >> to be > > > >> > available in a few days as rollout completes. > > > >> > > > > >> > > > > >> > > > > >> > On Wed, Aug 8, 2018 at 9:10 AM Himanshu <g.himan...@gmail.com> > > wrote: > > > >> > > > > >> > > +1 , thanks for coordinating it. > > > >> > > > > > >> > > On Tue, Aug 7, 2018 at 8:05 PM, Gian Merlino <g...@apache.org> > > > wrote: > > > >> > > > > > >> > > > +1. Thank you Jihoon for running this release. > > > >> > > > > > > >> > > > On Tue, Aug 7, 2018 at 10:04 AM Jihoon Son < > > jihoon...@apache.org> > > > >> > wrote: > > > >> > > > > > > >> > > > > Sure, > > > >> > > > > > > > >> > > > > the release note is available here: > > > >> > > > > https://github.com/apache/incubator-druid/issues/6116. > > > >> > > > > > > > >> > > > > Best, > > > >> > > > > Jihoon > > > >> > > > > > > > >> > > > > On Tue, Aug 7, 2018 at 10:02 AM Charles Allen < > > > cral...@apache.org > > > >> > > > > >> > > > wrote: > > > >> > > > > > > > >> > > > > > ((don't let this ask block the release)) > > > >> > > > > > > > > >> > > > > > Is there a way to get a preview of what the release notice > > > will > > > >> > look > > > >> > > > > like? > > > >> > > > > > > > > >> > > > > > On Mon, Aug 6, 2018 at 3:38 PM Fangjin Yang < > > fang...@imply.io > > > > > > > >> > > wrote: > > > >> > > > > > > > > >> > > > > > > +1 > > > >> > > > > > > > > > >> > > > > > > On Mon, Aug 6, 2018 at 3:03 PM, Jihoon Son < > > > >> jihoon...@apache.org > > > >> > > > > > >> > > > > wrote: > > > >> > > > > > > > > > >> > > > > > > > Hi all, > > > >> > > > > > > > > > > >> > > > > > > > Druid 0.12.2-rc1 ( > > > >> > > > > > >> > > > > >> > > > > > > https://urldefense.proofpoint.com/v2/url?u=http-3A__druid.io_downloads.html&d=DwIBaQ&c=ncDTmphkJTvjIDPh0hpF_w&r=HrLGT1qWNhseJBMYABL0GFSZESht5gBoLejor3SqMSo&m=jpC6qqjxYOq5XtNdLQ8U99CX0aj4Qn7oNQCLnjJ_v-Y&s=wdtYVmGEMAXuXYDvVCvJ_u6CKSIShdGKA2DpgjX9lfE&e= > > > >> > ) > > > >> > > is available > > > >> > > > now, > > > >> > > > > > and > > > >> > > > > > > I > > > >> > > > > > > > think it's time to vote on the 0.12.2 release. Please > > note > > > >> that > > > >> > > > > 0.12.2 > > > >> > > > > > is > > > >> > > > > > > > not an ASF release. > > > >> > > > > > > > > > > >> > > > > > > > Here is my +1. > > > >> > > > > > > > > > > >> > > > > > > > Best, > > > >> > > > > > > > Jihoon > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > > > > >