> On Dec. 3, 2012, 11:42 a.m., Ivan Kelly wrote: > > hedwig-server/src/main/java/org/apache/hedwig/server/meta/MsMetadataManagerFactory.java, > > line 511 > > <https://reviews.apache.org/r/8038/diff/2/?file=189059#file189059line511> > > > > Where do these constants come from? > > Jiannan Wang wrote: > For subscription data, concat "topic" and "subId" as a key in MetaStore, > and the we can scan the topic to get the subscriber list. > These constants are used as separator for "topic" and "subId". > > Ivan Kelly wrote: > why did you choose '\001' and '\002' though? It seems a bit random.
Yeah, we choose it at random, any char that will not being used in topic and subId is Okay for it. What do you suggest? - Jiannan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8038/#review13965 ----------------------------------------------------------- On Nov. 13, 2012, 1:51 p.m., Jiannan Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8038/ > ----------------------------------------------------------- > > (Updated Nov. 13, 2012, 1:51 p.m.) > > > Review request for bookkeeper. > > > Description > ------- > > Implement a meta store based hedwig metadata manager. > > > This addresses bug BOOKKEEPER-262. > https://issues.apache.org/jira/browse/BOOKKEEPER-262 > > > Diffs > ----- > > > hedwig-server/src/main/java/org/apache/hedwig/server/common/ServerConfiguration.java > 520744a > > hedwig-server/src/main/java/org/apache/hedwig/server/meta/MsMetadataManagerFactory.java > PRE-CREATION > > hedwig-server/src/test/java/org/apache/hedwig/server/meta/MetadataManagerFactoryTestCase.java > e33b315 > > Diff: https://reviews.apache.org/r/8038/diff/ > > > Testing > ------- > > > Thanks, > > Jiannan Wang > >
