> On 2012-02-21 10:06:21, Sijie Guo wrote: > > bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/FileSystemUpgrade.java, > > line 68 > > <https://reviews.apache.org/r/3910/diff/1/?file=75146#file75146line68> > > > > should it be a constant define?
No, unless we define it as LAYOUT_VERSION_1 which kinda defeats the purpose. I never what this number to be changed, so having a magic number here is fine in my opinion. - Ivan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3910/#review5238 ----------------------------------------------------------- On 2012-02-15 14:08:46, Ivan Kelly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3910/ > ----------------------------------------------------------- > > (Updated 2012-02-15 14:08:46) > > > Review request for bookkeeper. > > > Summary > ------- > > Part of BOOKKEEPER-163. This improvement creates a framework to allow > administrators to upgrade the filesystem layout of previous bookkeeper > versions to be usable by the most recent software. The upgrade processes are > currently empty until BOOKKEEPER-163 adds something. > > > This addresses bug BOOKKEEPER-172. > https://issues.apache.org/jira/browse/BOOKKEEPER-172 > > > Diffs > ----- > > bookkeeper-server/bin/bookkeeper b0629bd > > bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/FileSystemUpgrade.java > PRE-CREATION > > bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/UpgradeTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/3910/diff > > > Testing > ------- > > > Thanks, > > Ivan > >
