Hi Dave, Thank you for your inputs. To give more context, the reason we are doing the release is to fix a backwards incompatible change with the save_params API. This issue impacted some of our users and the warning message in RC0 showed an alternative API for one use case but not the other when save_params was called. We are fixing it to provide a warning message for both use cases, so that users may not end up replacing save_params with the wrong API. My sincere apologies to the mentors, for creating additional work.
Anirudh On Mon, Jul 2, 2018, 6:44 PM Dave Fisher <dave2w...@comcast.net> wrote: > Mentors all - Releases are not cheap and we are all volunteers. > > Can we teach Release Managers to have a little more judgement about when > to pull a release? > > It seems to be too risk adverse to cancel this vote due to a warning > message. > > My 2 cents. > > Regards, > Dave > > Sent from my iPhone > > > On Jul 2, 2018, at 5:45 PM, Anirudh <anirudh2...@gmail.com> wrote: > > > > Hi, > > > > We are cancelling this vote because we need to improve the warning > message > > when users use a deprecated API: `save_params`. We will address the issue > > and send out a new release for another vote. > > > > Thanks, > > > > Anirudh > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >
