The same problem still exists with @GET file/{objectId}
buildGetMediaResponse(id, objectEntry) where objectEntry has dead
refferences from a previous spi session.

On Mon, Dec 21, 2009 at 10:13 AM, Flo <[email protected]> wrote:
> Thanks,
>
> Works for me.
>
> Chears
>
> Florian Roth
>
> On Fri, Dec 18, 2009 at 7:20 PM, Florent Guillaume <[email protected]> wrote:
>> Ok I think I have fixed this, please check now.
>> Cf https://issues.apache.org/jira/browse/CMIS-84
>>
>> Florent
>>
>>
>> On Mon, Dec 14, 2009 at 2:45 PM, Florian Roth <[email protected]> 
>> wrote:
>>> Hi,
>>>
>>> this is happening because you get a node in one session close the session, 
>>> node is still in memory, then you open new spi session and want to fetch 
>>> properties of the in-memory node from the previous session.
>>> All this has to be done in one session. So I think opening (only) one 
>>> session for every request is good. It's fixed with a protected SPI in 
>>> CMISObjectCollection. And use this spi in every subcollection. So you are 
>>> save that all operations in one request are done in one session, and you 
>>> don't have to take care to close the spi session in every method.
>>>
>>> Best regards
>>>
>>>
>>> -----Ursprüngliche Nachricht-----
>>> Von: Florent Guillaume [mailto:[email protected]]
>>> Gesendet: Montag, 14. Dezember 2009 14:22
>>> An: chemistry-dev
>>> Betreff: Re: AW: AW: atompub jcr bug with last commit
>>>
>>> Hi folks,
>>>
>>> Regarding these "javax.jcr.RepositoryException: this session has been
>>> closed" problems, I would like to fix it but need a unit test to know
>>> when it happens and to know what we're dealing with here. Can anyone
>>> provide that?
>>>
>>> Florent
>>>
>>>
>>> On Mon, Dec 7, 2009 at 4:11 PM, Florian Roth <[email protected]> 
>>> wrote:
>>>> Very cool. I will try this these days.
>>>>
>>>> Best regards
>>>>
>>>> -----Ursprüngliche Nachricht-----
>>>> Von: Serge Huber [mailto:[email protected]]
>>>> Gesendet: Montag, 7. Dezember 2009 16:09
>>>> An: [email protected]
>>>> Betreff: Re: AW: AW: atompub jcr bug with last commit
>>>>
>>>>
>>>> Hi Florian,
>>>>
>>>> I have just created CMIS-73 (https://issues.apache.org/jira/browse/CMIS-73
>>>> ). Hopefully this patch will work properly. I did it against the trunk.
>>>>
>>>> Regards,
>>>>   Serge Huber.
>>>>
>>>> On 7 déc. 09, at 15:39, Florian Roth wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> it would be very nice if you could add an patch in JIRA with your
>>>>> jcr test cases. I wanted to write some too. But if you have some.
>>>>> Its great.
>>>>> I don't know if somebody is developing at the chemistry-jcr
>>>>> subproject but I think many are interested in it.
>>>>>
>>>>> Best regards
>>>>>
>>>>> Florian
>>>>>
>>>>> -----Ursprüngliche Nachricht-----
>>>>> Von: Serge Huber [mailto:[email protected]]
>>>>> Gesendet: Montag, 7. Dezember 2009 15:32
>>>>> An: [email protected]
>>>>> Betreff: Re: AW: atompub jcr bug with last commit
>>>>>
>>>>>
>>>>> Hi Florian,
>>>>>
>>>>> Thanks, I could send out a patch attached to a JIRA, or just out to
>>>>> the list (not sure it would like the attachement though), or privately
>>>>> to you if you want to have a look.
>>>>>
>>>>> Regards,
>>>>>   Serge Huber.
>>>>>
>>>>> On 7 déc. 09, at 15:21, Florian Roth wrote:
>>>>>
>>>>>> Hi Serge,
>>>>>>
>>>>>> Thank you.
>>>>>>
>>>>>> My way is a protected SPI in CMISObjectsCollection.
>>>>>> Spi=repository.getSPI() in constructor and a destructor with
>>>>>> spi.close() it works. But don't know if that is a good way.
>>>>>>
>>>>>> I am very interested in jcr tests
>>>>>>
>>>>>> Chears Florian
>>>>>>
>>>>>>
>>>>>> -----Ursprüngliche Nachricht-----
>>>>>> Von: Serge Huber [mailto:[email protected]]
>>>>>> Gesendet: Montag, 7. Dezember 2009 14:37
>>>>>> An: [email protected]
>>>>>> Betreff: Re: atompub jcr bug with last commit
>>>>>>
>>>>>>
>>>>>> Hi Florian,
>>>>>>
>>>>>> I have seen the same thing, I sorta modified CMISChildrenCollection
>>>>>> to
>>>>>> get around a few, but I must say I'm no expert in this area as I am
>>>>>> just starting. Basically what I did was put the calls to the SPI
>>>>>> inside getFeed which is the only public method and all protected
>>>>>> methods that need references will get the SPI as a parameter.
>>>>>>
>>>>>> Regards,
>>>>>>  Serge Huber.
>>>>>>
>>>>>> On 7 déc. 09, at 14:12, Florian Roth wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> the commit from 02.12.2009 [886201] makes problems with chemistry-
>>>>>>> jcr. Something is wrong with Session closing (spi.close()). Got lots
>>>>>>> of javax.jcr.RepositoryException: this session has been closed
>>>>>>> exceptions.
>>>>>>>
>>>>>>>
>>>>>>> Chears
>>>>>>>
>>>>>>> in-integrierte informationssysteme GmbH
>>>>>>> Am Seerhein 8 . 78467 Konstanz
>>>>>>> http://www.in-gmbh.de/ . mailto:[email protected]
>>>>>>>
>>>>>>> Telefon: +49 7531 8145-0
>>>>>>> Telefax: +49 7531 8145-81
>>>>>>>
>>>>>>>
>>>>>>> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
>>>>>>> Informationen. Wenn Sie nicht der richtige Adressat sind oder diese
>>>>>>> E-Mail irrtümlich erhalten haben, informieren Sie bitte sofort den
>>>>>>> Absender und vernichten Sie diese Mail. Das unerlaubte Kopieren
>>>>>>> sowie die unbefugte Weitergabe dieser Mail sind nicht gestattet.
>>>>>>> This e-mail may contain confidential and/or privileged information.
>>>>>>> If you are not the intended recipient (or have received this e-mail
>>>>>>> in error) please notify the sender immediately and destroy this e-
>>>>>>> mail. Any unauthorised copying, disclosure or distribution of the
>>>>>>> material in this e-mail is strictly forbidden.
>>>>>>>
>>>>>>> Sitz der Gesellschaft: Konstanz . Amtsgericht Freiburg i.B. .
>>>>>>> Registernummer: HRB 380976 . Geschäftsführer: Siegfried Wagner
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>>
>>> --
>>> Florent Guillaume, Director of R&D, Nuxeo
>>> Open Source, Java EE based, Enterprise Content Management (ECM)
>>> http://www.nuxeo.com   http://www.nuxeo.org   +33 1 40 33 79 87
>>>
>>
>>
>>
>> --
>> Florent Guillaume, Director of R&D, Nuxeo
>> Open Source, Java EE based, Enterprise Content Management (ECM)
>> http://www.nuxeo.com   http://www.nuxeo.org   +33 1 40 33 79 87
>>
>

Reply via email to