[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Druid 0.12.2 release vote


Awesome! Thanks Charles!

Jihoon

On Wed, Aug 8, 2018 at 1:16 PM Gian Merlino <gian@xxxxxxxxxx> wrote:

> Thanks, it will be nice to see!
>
> On Wed, Aug 8, 2018 at 1:15 PM Charles Allen <charles.allen@xxxxxxxx
> .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 <gian@xxxxxxxxxx> 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 <gian@xxxxxxxxxx> 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 <jihoonson@xxxxxxxxxx>
> > 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.allen@xxxxxxxx.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.himanshu@xxxxxxxxx>
> > wrote:
> > > >> >
> > > >> > > +1 , thanks for coordinating it.
> > > >> > >
> > > >> > > On Tue, Aug 7, 2018 at 8:05 PM, Gian Merlino <gian@xxxxxxxxxx>
> > > wrote:
> > > >> > >
> > > >> > > > +1. Thank you Jihoon for running this release.
> > > >> > > >
> > > >> > > > On Tue, Aug 7, 2018 at 10:04 AM Jihoon Son <
> > jihoonson@xxxxxxxxxx>
> > > >> > 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 <
> > > crallen@xxxxxxxxxx
> > > >> >
> > > >> > > > 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 <
> > fangjin@xxxxxxxx
> > > >
> > > >> > > wrote:
> > > >> > > > > >
> > > >> > > > > > > +1
> > > >> > > > > > >
> > > >> > > > > > > On Mon, Aug 6, 2018 at 3:03 PM, Jihoon Son <
> > > >> jihoonson@xxxxxxxxxx
> > > >> > >
> > > >> > > > > 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
> > > >> > > > > > > >
> > > >> > > > > > >
> > > >> > > > > >
> > > >> > > > >
> > > >> > > >
> > > >> > >
> > > >> >
> > > >>
> > > >
> > >
> >
>