Re: Review Request 11299: configurable to support different SHA algorithm

2013-07-18 Thread Rich Thompson
On May 22, 2013, 12:55 a.m., Stanton Sievers wrote: http://svn.apache.org/repos/asf/shindig/trunk/java/common/conf/shindig.properties, lines 228-229 https://reviews.apache.org/r/11299/diff/2/?file=295178#file295178line228 It is not clear to me why these are lists. The code that

Re: Review Request 11299: configurable to support different SHA algorithm

2013-07-18 Thread Zhi Hong Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11299/ --- (Updated July 18, 2013, 11:04 a.m.) Review request for shindig, Dan Dumont,

AW: Review Request: Alternative database backend based on graph database neo4j

2013-07-18 Thread René Peinl
Dear Ryan, fair question. The main contribution is to allow usage of neo4j as an alternative database backend which improves performance significantly in some areas compared to ORM and MySQL (see http://www.edbt.org/Proceedings/2013-Genova/papers/workshops/a29-holzschuher .pdf). In addition to

Re: Review Request: Alternative database backend based on graph database neo4j

2013-07-18 Thread Ryan Baxter
Great. So what technically will the code that is contributed to Shindig do? How does it talk to the neo4j backend? On Thu, Jul 18, 2013 at 9:25 AM, René Peinl rene.pe...@hof-university.de wrote: Dear Ryan, fair question. The main contribution is to allow usage of neo4j as an alternative

Re: Review Request 12647: Replacement for iGoogle gadgetsettings

2013-07-18 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12647/#review23438 --- In general I understand what is happening here but I feel like

Re: Replacement for iGoogle gadgetsettings

2013-07-18 Thread Ryan Baxter
On Wed, Jul 17, 2013 at 6:20 AM, Charl van Niekerk c.vanniek...@utwente.nl wrote: On Tue, Jul 09, 2013 at 12:19:13PM -0400, Ryan Baxter wrote: Sounds like a good enough spot to me. I added the following classes: org.apache.shindig.gadgets.servlet.GadgetSettingsServlet

Re: Review Request 11299: configurable to support different SHA algorithm

2013-07-18 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11299/#review23441 --- Can you add a JIRA to the review? - Ryan Baxter On July 18,

Re: Review Request 11299: configurable to support different SHA algorithm

2013-07-18 Thread Ryan Baxter
On May 22, 2013, 12:55 a.m., Stanton Sievers wrote: http://svn.apache.org/repos/asf/shindig/trunk/java/common/conf/shindig.properties, lines 228-229 https://reviews.apache.org/r/11299/diff/2/?file=295178#file295178line228 It is not clear to me why these are lists. The code that

Re: Shindig Assembly On builds.a.o

2013-07-18 Thread Ryan Baxter
I looked at the rave-project-trunk job in Jenkins on builds.a.o and there is a Deploy Artifacts To Maven Repository post build action but it is not configured, so it doesn't look like it is. On Tue, Jul 16, 2013 at 10:51 PM, Matt Franklin m.ben.frank...@gmail.com wrote: Jenkins should have the

Re: Review Request 11299: configurable to support different SHA algorithm

2013-07-18 Thread Zhi Hong Yang
On July 18, 2013, 9:16 p.m., Ryan Baxter wrote: Can you add a JIRA to the review? Here is the JIRA Link, please help to check it's ok? thanks. https://issues.apache.org/jira/browse/SHINDIG-1921 - Zhi Hong --- This is an

Re: Shindig Assembly On builds.a.o

2013-07-18 Thread Matt Franklin
There shouldn't be any config. That job deploys our snapshots On Jul 18, 2013, at 16:19, Ryan Baxter rbaxte...@gmail.com wrote: I looked at the rave-project-trunk job in Jenkins on builds.a.o and there is a Deploy Artifacts To Maven Repository post build action but it is not configured, so