Closed sqlalchemy-bot closed 18 years ago
Michael Bayer (zzzeek) wrote:
this bug is sort of dead for now. reopen if theres any reported issues with options(), its been working fine.
Michael Bayer (zzzeek) wrote:
yes, the MapperOption would have to detect the list and spawn out a list of EagerLazyOption objects, and use those.
Michael Bayer (zzzeek) wrote:
this was improved a bunch of changesets ago when the "mapper singleton" behavior was removed for the more lightweight way it is now. still should go thorugh and see if it really works and that its not making lots of excess objects (or maybe it is and who cares). so bumping priority down.
Anonymous wrote:
(original author: Justin) could the syntax be
mapper.options(eagerload(['foo','bar','foo.bar','bar.lala']('foo','bar','foo.bar','bar.lala')))
Changes by Michael Bayer (zzzeek): set state to "wontfix"
Changes by Michael Bayer (zzzeek): removed "critical" priority
Changes by Michael Bayer (zzzeek): set priority to "critical"
Issue created by Michael Bayer (zzzeek)
tried an example like this:
mapper.options(eagerload('foo'), eagerload('bar'), eagerload('foo.bar'), eagerload('bar.lala'), ...)
it didnt seem to work, and even if it did Im not completely sure that its not creating properties and then throwing them away, etc. dig into this and see how its really working, look into smarter ways (like checking each properties parent mapper before deciding to copy it)