Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread ๏̯͡๏ Jasvir Nagra
On Wed, Oct 19, 2011 at 5:01 PM, Ryan J Baxter wrote: > Jas does the cached entries expire at some point? I assume this would be > another configuration option... > -Ryan > I am not sure. Eyeballing the calls shindig's css parser and sanitizer make into Caja bit more, I notice that the message

Re: Review Request: Declarative action does not work on IE

2011-10-19 Thread Ryan Baxter
> On 2011-10-20 00:13:57, Ryan Baxter wrote: > > Committed revision 1186576. Please close the review and the JIRA - Ryan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2404/#review2687 -

Re: Review Request: Declarative action does not work on IE

2011-10-19 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2404/#review2687 --- Ship it! Committed revision 1186576. - Ryan On 2011-10-19 20:10:22

Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread Ryan J Baxter
Jas does the cached entries expire at some point? I assume this would be another configuration option... -Ryan Email: rjbax...@us.ibm.com Phone: 978-899-3041 developerWorks Profile From: ๏̯͡๏ Jasvir Nagra To: dev@shindig.apache.org, Date: 10/19/2011 07:40 PM Subject:Re: Memo

Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread ๏̯͡๏ Jasvir Nagra
A css rule-set is what just a css selector (or more accurately, one or more css selectors) and a block of css declarations. Eg: div { background-color: red; color: pink; } is a rule-set. It depends on what you're parsing - typically, a stylesheet, webpage or gadget will have tens of rules and if

Re: Review Request: Security Token Cleanup in preparation for ContainerConfig controlled token expirations

2011-10-19 Thread Matt Marum
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2396/#review2676 --- Just some nitpicks. I'm good with the changes. I wish I knew if th

Re: Regarding Open Social 2.0 Implementation in Shindig

2011-10-19 Thread Eric Woods
Hi Lalaji, I'm the primary implementor of Activity Streams in Apache Shindig. All of your questions are good. I'll do my best to address each one: 1) There is no intentional mapping between Activity Streams and OpenSocial's "Activity" construct. The "Activity" construct largely considered a

Re: Review Request: Declarative action does not work on IE

2011-10-19 Thread Jason Chiang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2404/ --- (Updated 2011-10-19 20:10:22.587736) Review request for shindig, Ryan Baxter and

Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread Stanton Sievers
Thanks Jas. The parsed css cache is the one I was referring to. EhCache is the cache provider in this case, so that's probably why I was seeing EhCache references to the Caja objects in the Eclipse MAT. I'll disable it and see what I can find. In the meantime, would 1700 of these RuleSets se

Re: Review Request: Security Token Cleanup in preparation for ContainerConfig controlled token expirations

2011-10-19 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2396/ --- (Updated 2011-10-19 19:46:39.109455) Review request for shindig, Matt Marum, Rya

Re: Review Request: Declarative action does not work on IE

2011-10-19 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2404/#review2674 --- http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javas

Re: Review Request: Security Token Cleanup in preparation for ContainerConfig controlled token expirations

2011-10-19 Thread Jesse Ciancetta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2396/#review2673 --- Ship it! Found a few more small items -- noted them in the review --

Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread ๏̯͡๏ Jasvir Nagra
And he hangs out here. :) As you'd expect, CssTree.RuleSets (and all the other parse tree nodes) are retained as long as the root of the parse tree is retained. Nothing in the Caja parser retains a reference to the root of the parse tree after the parse. In Shindig, shindig/java/gadgets/src/mai

Re: listMethods and https

2011-10-19 Thread daviesd
Dan, thanks for the response. 1) I changed it to https:// because at the time we took a snapshot this was not working. I suspect it DOES work now and I can revert it back. 2) We determined this is only going to be an issue for self-signed certificates (in our QA environment). We are guessing in

Re: listMethods and https

2011-10-19 Thread Dan Dumont
Well.. you could try to set it to localhost and then provide custom values in the CommonContainer config. Wondering why you changed this to https:// instead of // Starting a url with // should use the protocol being used by the page. was it not working for https in your case? I'm not sure why

Re: Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread Ryan J Baxter
You want to ask Jasvir, he is the Caja lead at Google :) -Ryan Email: rjbax...@us.ibm.com Phone: 978-899-3041 developerWorks Profile From: Stanton Sievers/Westford/IBM@Lotus To: dev@shindig.apache.org, Date: 10/19/2011 02:21 PM Subject:Memory concerns: what is the "normal" amo

Re: Review Request: Security Token Cleanup in preparation for ContainerConfig controlled token expirations

2011-10-19 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2396/ --- (Updated 2011-10-19 18:40:11.950651) Review request for shindig, Matt Marum, Rya

Re: Review Request: Security Token Cleanup in preparation for ContainerConfig controlled token expirations

2011-10-19 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2396/ --- (Updated 2011-10-19 18:38:20.724229) Review request for shindig, Matt Marum, Rya

Memory concerns: what is the "normal" amount of CssTree.RuleSet objects to see at runtime

2011-10-19 Thread Stanton Sievers
Hi everyone, I'm seeing a lot of com.google.caja.parser.css.CssTree$RuleSet objects when running Shindig. I'm seeing around 1700 of these RuleSet objects being retained, which equates to between 9MB and 10MB. This constitutes ~15% of the total heap size! These RuleSets appear to be in an Arr

Re: Review Request: Containers should be able to override a subset of the rpc handlers of the actions feature.

2011-10-19 Thread Ryan Baxter
> On 2011-10-19 01:38:51, Ryan Baxter wrote: > > LGTM. Matt please create a JIRA and attach the patch to it. I found the JIRA. Committed revision 1186350. - Ryan --- This is an automatically generated e-mail. To reply, visit: https://r

Re: listMethods and https

2011-10-19 Thread daviesd
*bump* The last time I posted this message it didn¹t gain any traction. However this is now affecting us in a production environment and forces us to import the certificate of the load balancer into our shindig server. Ideas? On 6/1/11 3:33 PM, "Davies,Douglas" wrote: > Our shindig server ru