- wrong! I didn't have the cprocs on the master test. we are barely

slower.   we do save on single queries, however.
- but...wrong!  we still can't do this approach for eager loads.
we don't have context.outer_adapter available when we set up
"quick" here, so this can't work.

1 files changed, 5 insertions, 0 deletions

diff --git a/lib/sqlalchemy/orm/strategies.py b/lib/sqlalchemy/orm/strategies.py
index 95ed198a5..41f678955 100644
--- a/lib/sqlalchemy/orm/strategies.py
+++ b/lib/sqlalchemy/orm/strategies.py
@@ -148,6 +148,11 @@ class ColumnLoader(LoaderStrategy):
             # the column on the base table.   More specific logic
             # should be added here so that we need not bother
             # querying out every column.
+
+        # TODO: wrong.  we are again fetching by column and we are
+        # not applying context.outer_adapter if we're in the primary
+        # collection.  Also, context.outer_adapter
+        # isn't even set up yet.  So again, we can't do this.
         populators["quick"].append(
             (self.key, operator.itemgetter(self.columns[0]))
         )