Ive committed this fix in r3425.  remove() calls close() on an  
existing session if one is present.


On Aug 29, 2007, at 6:49 PM, jason kirtland wrote:

>
> Jian wrote:
>>
>> the remove is described in doc as close + extra cleanup
>> Shouldn't the doc be updated to add that point, which will make
>> the the life of newbies like me easier =D
>> Or better: add the explicitly close in remove.
>
> Yeah, the current remove() implementation in trunk isn't yet in
> sync with the docs.  I'm not sure if it is simply not complete or
> if the feature is in flux...  It will get resolved one way or
> another before the next beta.  But for now, I'm pretty confident
> that having that explicit close() in there won't hurt you going
> forward.
>
>
> >


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To post to this group, send email to sqlalchemy@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sqlalchemy?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to