diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2022-06-30 19:10:06 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2022-07-01 12:14:02 -0400 |
| commit | 741af02893a19c879ba4d929151b9358aeb48148 (patch) | |
| tree | 7b408bb82033d90cff4ba9e78ead2c0cda27411f /lib/sqlalchemy/orm/context.py | |
| parent | 286e5fb649f77367883800ba4ec3d536e8031ca8 (diff) | |
| download | sqlalchemy-741af02893a19c879ba4d929151b9358aeb48148.tar.gz | |
repair yield_per for non-SS dialects and add new options
Implemented new :paramref:`_engine.Connection.execution_options.yield_per`
execution option for :class:`_engine.Connection` in Core, to mirror that of
the same :ref:`yield_per <orm_queryguide_yield_per>` option available in
the ORM. The option sets both the
:paramref:`_engine.Connection.execution_options.stream_results` option at
the same time as invoking :meth:`_engine.Result.yield_per`, to provide the
most common streaming result configuration which also mirrors that of the
ORM use case in its usage pattern.
Fixed bug in :class:`_engine.Result` where the usage of a buffered result
strategy would not be used if the dialect in use did not support an
explicit "server side cursor" setting, when using
:paramref:`_engine.Connection.execution_options.stream_results`. This is in
error as DBAPIs such as that of SQLite and Oracle already use a
non-buffered result fetching scheme, which still benefits from usage of
partial result fetching. The "buffered" strategy is now used in all
cases where :paramref:`_engine.Connection.execution_options.stream_results`
is set.
Added :meth:`.FilterResult.yield_per` so that result implementations
such as :class:`.MappingResult`, :class:`.ScalarResult` and
:class:`.AsyncResult` have access to this method.
Fixes: #8199
Change-Id: I6dde3cbe483a1bf81e945561b60f4b7d1c434750
Diffstat (limited to 'lib/sqlalchemy/orm/context.py')
| -rw-r--r-- | lib/sqlalchemy/orm/context.py | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/lib/sqlalchemy/orm/context.py b/lib/sqlalchemy/orm/context.py index a468244e9..e743eeed5 100644 --- a/lib/sqlalchemy/orm/context.py +++ b/lib/sqlalchemy/orm/context.py @@ -357,14 +357,9 @@ class ORMCompileState(CompileState): else: execution_options = execution_options.union(_orm_load_exec_options) - if "yield_per" in execution_options or load_options._yield_per: + if load_options._yield_per: execution_options = execution_options.union( - { - "stream_results": True, - "max_row_buffer": execution_options.get( - "yield_per", load_options._yield_per - ), - } + {"yield_per": load_options._yield_per} ) if ( |