diff options
| author | Heckad <heckad@yandex.ru> | 2020-01-01 14:47:01 -0500 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2020-01-03 13:09:46 -0500 |
| commit | a076b1f30406cbb59a55e2c01ddd17a84636778e (patch) | |
| tree | bb4c5a5b88d45963040842427d73f9616465bdb0 /lib/sqlalchemy/ext/automap.py | |
| parent | 748bf710fdeeed327933a5e7c277c7f79f6053a6 (diff) | |
| download | sqlalchemy-a076b1f30406cbb59a55e2c01ddd17a84636778e.tar.gz | |
Use context managers for threading.Lock()
(zzzeek:) For some reason I thought that threading.Lock() still did
not support context managers, at least in Python 2, however this
does not seem to be the case.
Co-authored-by: Mike Bayer <mike_mp@zzzcomputing.com>
Closes: #5069
Pull-request: https://github.com/sqlalchemy/sqlalchemy/pull/5069
Pull-request-sha: efeac06dda5afdbe33abcf9b27c8b5b5725c8444
Change-Id: Ic64fcd99cd587bc70b4ecc5b45d8205b5c76eff2
Diffstat (limited to 'lib/sqlalchemy/ext/automap.py')
| -rw-r--r-- | lib/sqlalchemy/ext/automap.py | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/lib/sqlalchemy/ext/automap.py b/lib/sqlalchemy/ext/automap.py index 5fa4e8822..24c5117c8 100644 --- a/lib/sqlalchemy/ext/automap.py +++ b/lib/sqlalchemy/ext/automap.py @@ -762,8 +762,7 @@ class AutomapBase(object): autoload_replace=False, ) - _CONFIGURE_MUTEX.acquire() - try: + with _CONFIGURE_MUTEX: table_to_map_config = dict( (m.local_table, m) for m in _DeferredMapperConfig.classes_for_base( @@ -818,8 +817,6 @@ class AutomapBase(object): for map_config in _DeferredMapperConfig.classes_for_base(cls): map_config.map() - finally: - _CONFIGURE_MUTEX.release() _sa_decl_prepare = True """Indicate that the mapping of classes should be deferred. |