diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2017-07-13 18:32:42 -0400 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2017-08-07 15:05:00 -0400 |
commit | 68879d50faa9e2602e55d5d191647b1cf864e5ab (patch) | |
tree | be9f9e906a3674aa7237ae564eee244931399bae /lib/sqlalchemy/orm/loading.py | |
parent | 4b4f8fbf25f1a5a76c1579c1a3fd6ffad07c8c66 (diff) | |
download | sqlalchemy-68879d50faa9e2602e55d5d191647b1cf864e5ab.tar.gz |
Enable multi-level selectin polymorphic loading
Change-Id: Icc742bbeecdb7448ce84caccd63e086af16e81c1
Fixes: #4026
Diffstat (limited to 'lib/sqlalchemy/orm/loading.py')
-rw-r--r-- | lib/sqlalchemy/orm/loading.py | 27 |
1 files changed, 18 insertions, 9 deletions
diff --git a/lib/sqlalchemy/orm/loading.py b/lib/sqlalchemy/orm/loading.py index 48c0db851..e4aea3994 100644 --- a/lib/sqlalchemy/orm/loading.py +++ b/lib/sqlalchemy/orm/loading.py @@ -360,20 +360,26 @@ def _instance_processor( if ( key in context.attributes and context.attributes[key].strategy == - (('selectinload_polymorphic', True), ) and - mapper in context.attributes[key].local_opts['mappers'] - ) or mapper.polymorphic_load == 'selectin': + (('selectinload_polymorphic', True), ) + ): + selectin_load_via = mapper._should_selectin_load( + context.attributes[key].local_opts['entities'], + _polymorphic_from) + else: + selectin_load_via = mapper._should_selectin_load( + None, _polymorphic_from) + if selectin_load_via and selectin_load_via is not _polymorphic_from: # only_load_props goes w/ refresh_state only, and in a refresh # we are a single row query for the exact entity; polymorphic # loading does not apply assert only_load_props is None - callable_ = _load_subclass_via_in(context, path, mapper) + callable_ = _load_subclass_via_in(context, path, selectin_load_via) PostLoad.callable_for_path( - context, load_path, mapper, - callable_, mapper) + context, load_path, selectin_load_via, + callable_, selectin_load_via) post_load = PostLoad.for_context(context, load_path, only_load_props) @@ -523,12 +529,15 @@ def _instance_processor( return _instance -@util.dependencies("sqlalchemy.ext.baked") -def _load_subclass_via_in(baked, context, path, mapper): +def _load_subclass_via_in(context, path, entity): + mapper = entity.mapper zero_idx = len(mapper.base_mapper.primary_key) == 1 - q, enable_opt, disable_opt = mapper._subclass_load_via_in + if entity.is_aliased_class: + q, enable_opt, disable_opt = mapper._subclass_load_via_in(entity) + else: + q, enable_opt, disable_opt = mapper._subclass_load_via_in_mapper def do_load(context, path, states, load_only, effective_entity): orig_query = context.query |