Ok, no worries, (and no rush on my part) its not too much effort for me to keep it updated at the moment. Let me know if you want it upto a particular revision.
Ian


On 5 May 2008, at 07:59, Cassie wrote:

--000e0cd29d344e4693044c764643
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

I think this is a good and necessary change (you convinced me by email long ago) and will commit it soon. I've just been enjoying ascension day + svn
commit has been all screwy lately.

So, continuing sounds good!

- Cassie


On Sat, May 3, 2008 at 6:32 PM, Ian Boston (JIRA) <[EMAIL PROTECTED]> wrote:


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

Ian Boston commented on SHINDIG-203:
------------------------------------

A question,

Should I continue with this patch ?
its getting a bit big at 220K.

Extract Interfaces from the Social Model to make it easier to Implements
Social Containers

--------------------------------------------------------------------- ---------------------

                Key: SHINDIG-203
URL: https://issues.apache.org/jira/browse/ SHINDIG-203
            Project: Shindig
         Issue Type: Improvement
         Components: RESTful API (Java)
           Reporter: Ian Boston
        Attachments: SHINDIG-203-3.patch


Currently the PersonService and other API's in the
org.apache.shindig.social package contain concrete classes in the form of Pojos to represent the model. Although this is fine, it does limit the
implementation strategy to those methods that can operate on PoJos by
reflection and building the pojos. For some this is Ok, where there use
GCLib or ApsectJ to enhance the Pojos so avoiding excessive Object GC
activity, but for those more direct ORM implementations, that already use class extension and a method to make the persistence simpler, concrete
classes in the model make it impossible to implement the social model
storage.
The alternative is to copy the object from the storage into the model,
and or add caching outside the storage space. This is quite a lot of code to do property, will replicate a significant amount of ORM functionality and will have an impact on the GC as more objects will be created and destroyed
during each request cycle.
Converting the API to interfaces (containing the necessary Enums) will
enable any implementation strategy to work (including things like
javax.jcr.Node ) and is minimal impact on those that want a Pojo, since they
can very quickly implement their own Pojos. It will also prevent the
consumer of the API creating object instances and assuming that they will be
saved.
I will attach a patch in a moment, and add a related implementation of
the Storage layer that uses interfaces.

--
This message is automaticall

Reply via email to