On Fri, Jun 01, 2007 at 07:17:19AM -0700, Michael Bayer wrote:
> so i didnt add filter() because i didnt feel like getting all the bug
> reports from people who have instance variables called "filter", and
> also because my plan was to do away with *all* the select/filter/etc
> methods and have everything go through query().

Right. I didn't think of that possibility. Do I read that as
"assign_mapper will die anyway" because you have an implicit "PITA" tag
on it already? ;)

> but if we dont think its a problem im certainly not going to hold back
> assign_mapper from what people want it to be, im just putting out the
> issues there and you all can decide.

I think I will refrain from using assign_mapper in the future. I'd like
to use .filter() because it's great and I'd rather use the explicit
mapper syntax instead of the (barely easier) assign_mapper syntax just
to be consistent. Sometimes using the assign_mapper because it's simpler
but in other places use the "mapper" methods because they are more
powerful doesn't really look consistent. If you asked me: either provide
these methods on the assign_mapper, too, or let the assign_mapper die
sooner or later. Am I right that we are just talking of

    john = session.query(User).get_by(name="john")

versus

    john = User.get_by(name="john")

here? I think I can live with that. Inserting/creating new objects looks
the same with both mapping methods AFAIK. So we are talking about
wasting 15 bytes in every query. I'll go waste some bytes then.

Cheers
 Christoph


--~--~---------~--~----~------------~-------~--~----~
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