[ 
https://issues.apache.org/jira/browse/SHINDIG-500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12622042#action_12622042
 ] 

John Hjelmstad commented on SHINDIG-500:
----------------------------------------

Hmmm.. bummer, though I'm glad we agree GadgetSpecs should be immutable, as 
that's my key goal here. As noted, as far as I can tell doing that requires 
caching of rewritten content (which I agree is of paramount importance) to 
occur somewhere other than (Basic)GadgetSpecFactory.

Your point about splitting/removing GadgetServer makes sense, but I'd like to 
find as incremental a change as possible to achieve GadgetSpec cleanup (closing 
in on immutability) in the meantime. The first thing that come to mind is to 
move the LruCache as written in BasicGadgetSpecFactory into GadgetServer (yes, 
making it even larger... perhaps an eventual forcing function for the change 
you're suggesting :)) to maintain caching semantics for rewritten content.

Assuming everyone using Shindig is also using BasicGadgetSpecFactory or a 
subclass of it, that ought to be acceptable (if notably suboptimal). But I have 
no idea if that's so -- thoughts?

> Make Gadget Object's content that of the active View
> ----------------------------------------------------
>
>                 Key: SHINDIG-500
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-500
>             Project: Shindig
>          Issue Type: Sub-task
>          Components: Gadget Rendering Server (Java)
>            Reporter: John Hjelmstad
>            Assignee: John Hjelmstad
>         Attachments: shindig-500.diff
>
>
> Step #1 of 
> http://mail-archives.apache.org/mod_mbox/incubator-shindig-dev/200808.mbox/[EMAIL
>  PROTECTED]

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to