----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11024/#review20841 -----------------------------------------------------------
I tried applying the patch to trunk but I am getting errors applying it. Can you double check that the patch is generated from trunk? - Ryan Baxter On May 21, 2013, 12:48 p.m., Venkat Mahadevan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11024/ > ----------------------------------------------------------- > > (Updated May 21, 2013, 12:48 p.m.) > > > Review request for shindig. > > > Description > ------- > > Currently the AppData is handling only a Map<String, String>. Need the > ability to handle values other than String, something like Map<String, > Object> so that we can have, for eg., boolean value as well. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java > 1481074 > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/AppDataService.java > 1481074 > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java > 1481074 > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java > 1481074 > > http://svn.apache.org/repos/asf/shindig/trunk/java/social-api/src/test/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialServiceTest.java > 1481074 > > Diff: https://reviews.apache.org/r/11024/diff/ > > > Testing > ------- > > > Thanks, > > Venkat Mahadevan > >
