Ah cool, thanks for the update, James! As for INFRA-11695, I could try if I have access to do it.
- Henry On Thu, May 12, 2016 at 2:44 PM, James Taylor <[email protected]> wrote: > We're waiting on INFRA for GitHub integration INFRA-11448, and JIRA setup > (INFRA-11445), and no ones sure how to do INFRA-11695 (maybe you know?) so > that INFRA can proceed with setting up svnpubsub for the website > (INFRA-11447). > > On Thu, May 12, 2016 at 2:34 PM, Henry Saputra <[email protected]> > wrote: > > > Shall we enable advance Github integration for Tephra ASF Git repo to > > enable PR for code review? > > > > Probably easier than reading diff online. > > > > - Henry > > > > On Thu, May 12, 2016 at 2:29 PM, Gokul Gunasekaran < > [email protected]> > > wrote: > > > > > Hi, > > > > > > I have copied couple of classes related to TransactionEdit which has > the > > > old cask package name for backward compatibility. > > > > > > I have also bumped up the TransactionLog current version to 3 which > > > represents logs written with the new apache package names. And the > > previous > > > versions are expected to be written with the old cask package names. > Now > > > the previous TransactionLogReaders expect the old cask package names > and > > > they convert it to new TransactionEdit (with apache package name) when > > they > > > return the read object. > > > > > > Also enhanced the unit test to make sure we can read the new > > > TransactionEdits (apache package) from logs written with old versions - > > 1, > > > 2 (cask package) as well as new version 3 (apache package). > > > > > > The old classes, as discussed before, will be removed in the upcoming > > > releases. > > > > > > Please review commit: > > > > > > > > > https://git1-us-west.apache.org/repos/asf?p=incubator-tephra.git;a=commit;h=afe31fb6 > > > > > > Thanks, > > > Gokul > > > > > >
