Re: Review Request: shindig atom API enhancements

2013-03-07 Thread Marshall Shi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9535/#review17598 --- - Call for review! - Marshall Shi On March 1, 2013, 3:33 a.m., Mar

Re: Review Request: gadget iframe title fix

2013-03-07 Thread Marshall Shi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9674/ --- (Updated March 8, 2013, 6:13 a.m.) Review request for shindig, Ryan Baxter, Dan

Re: Review Request: DefaultRequestPipeline should attempt conditional gets for stale, cached responses

2013-03-07 Thread Stanton Sievers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9528/ --- (Updated March 7, 2013, 9:40 p.m.) Review request for shindig. Changes ---

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

2013-03-07 Thread Henry Saputra
This is good news. One immediate comment is about the package name. Would it be possible to put it under org.apache.shindig rather than the de.hofuniversity? This would make the contributions uniform like from other companies and organizations. - Henry 2013/3/6 René Peinl > > ---

Re: Review Request: translation patch

2013-03-07 Thread Stanton Sievers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9603/#review17544 --- Ship it! Committed revision 1453901. Please close the review. - St

Re: Review Request: Do enhancement for FeatureRegistry to detect feature dependency loop problem

2013-03-07 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7497/#review17543 --- Ship it! Committed revision 1453895 - Ryan Baxter On March 4, 201

Re: Review Request: gadget iframe title fix

2013-03-07 Thread Ryan Baxter
> On March 7, 2013, 1:55 p.m., Ryan Baxter wrote: > > Committed please close the review I take that back. I had to revert the change because there are Javascript unit tests failing. Please reopen the review and fix the tests. - Ryan

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

2013-03-07 Thread Florian Holzschuher
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9773/#review17541 --- Hello, To activate the neo4j back-end you need to replace the follow

Re: Review Request: gadget iframe title fix

2013-03-07 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9674/#review17540 --- Ship it! Committed please close the review - Ryan Baxter On March

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

2013-03-07 Thread René Peinl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9773/ --- (Updated March 7, 2013, 1:17 p.m.) Review request for shindig. Changes ---

Re: Review Request: translation patch

2013-03-07 Thread Marshall Shi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9603/#review17534 --- Call for comments! - Marshall Shi On Feb. 26, 2013, 6:09 a.m., Mar

Re: Review Request: gadget iframe title fix

2013-03-07 Thread Marshall Shi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9674/#review17533 --- Ryan, any more comments? If not, please help to commit this change.