Alright, fair enough. This vote is now close. I'll revert CASSANDRA-5665 and restart a new vote with that.
-- Sylvain On Tue, Jun 25, 2013 at 9:12 AM, Jason Brown <jasedbr...@gmail.com> wrote: > Hi all, > > I finally dug into the dtest error on repair_test.TestRepair (in > > http://buildbot.datastax.com:8010/builders/cassandra-1.2/builds/513/steps/shell/logs/stdio > , > for example). Looking at the last successful run, and all the changes > since, I'm quite convinced the failure is due to a change I made in > Gossiper for CASSANDRA-5665. As this is the second problem with this patch, > I think we should revert it (as it's not mission critical), and proceed > with 1.2.6 without it. I will reevaluate the solution for the ticket this > week and consider how to best resolve. > > Apologies to all for delaying the release, and to Sylvain for the extra > building effort. > > Thanks, > > -Jason > > > > > On Sun, Jun 23, 2013 at 11:43 AM, Jason Brown <jasedbr...@gmail.com> > wrote: > > > Hey all, > > > > Before I vote, I'd like to spend Monday running some tests and basically > > beating the snot out of the changes I made last week. So please wait for > me > > before closing the vote. I should be within the 72 hours anyway, but I > want > > to give things another pass before we bless and ship. > > > > Thanks, > > > > -Jason > > > > > > On Sun, Jun 23, 2013 at 9:47 AM, Jonathan Ellis <jbel...@gmail.com> > wrote: > > > >> This is not a regression, so we won't be blocking the many other > >> improvements and fixes here for it. > >> On Jun 23, 2013 6:56 AM, "Cyril Scetbon" <cyril.scet...@free.fr> wrote: > >> > >> > -1 as we wait for CASSANDRA-5234 > >> > > >> > -- > >> > Cyril SCETBON > >> > > >> > On Jun 23, 2013, at 5:43 AM, Brandon Williams <dri...@gmail.com> > wrote: > >> > > >> > > +1 > >> > > > >> > > On Sat, Jun 22, 2013 at 5:46 AM, Sylvain Lebresne < > >> sylv...@datastax.com > >> > >wrote: > >> > > > >> > >> The previous assertion error has been fix so I propose the > following > >> > >> artifacts > >> > >> for release as 1.2.6. > >> > >> > >> > >> sha1: 043a63c63b92b0bf68047da6b5e379da35f3c688 > >> > >> Git: > >> > >> > >> > >> > >> > > >> > http://git-wip-us.apache.org/repos/asf?p=cassandra.git;a=shortlog;h=refs/tags/1.2.6-tentative > >> > >> Artifacts: > >> > >> > >> > >> > >> > > >> > https://repository.apache.org/content/repositories/orgapachecassandra-057/org/apache/cassandra/apache-cassandra/1.2.6/ > >> > >> Staging repository: > >> > >> > >> > > >> > https://repository.apache.org/content/repositories/orgapachecassandra-057/ > >> > >> > >> > >> The artifacts as well as the debian package are also available > here: > >> > >> http://people.apache.org/~slebresne/ > >> > >> > >> > >> The vote will be open for 72 hours (longer if needed). > >> > >> > >> > >> [1]: http://goo.gl/WcaX7 (CHANGES.txt) > >> > >> [2]: http://goo.gl/em5ZY (NEWS.txt) > >> > >> > >> > > >> > > >> > > > > >
