sqlalchemy-cubrid 1.4.1__tar.gz → 1.4.2__tar.gz

{sqlalchemy_cubrid-1.4.1/sqlalchemy_cubrid.egg-info → sqlalchemy_cubrid-1.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sqlalchemy-cubrid
-Version: 1.4.1
+Version: 1.4.2
 Summary: CUBRID dialect for SQLAlchemy
 Author-email: Yeongseon Choe <yeongseon.choe@gmail.com>, Gyeongjun Paik <paikend@gmail.com>
 License-Expression: MIT
@@ -45,7 +45,7 @@ Requires-Dist: alembic>=1.7; extra == "dev"
 Provides-Extra: alembic
 Requires-Dist: alembic>=1.7; extra == "alembic"
 Provides-Extra: pycubrid
-Requires-Dist: pycubrid<2.0,>=1.2.0; extra == "pycubrid"
+Requires-Dist: pycubrid<2.0,>=1.3.2; extra == "pycubrid"
 Dynamic: license-file
 
 # sqlalchemy-cubrid
@@ -202,7 +202,7 @@ async with AsyncSession(engine) as session:
 - **No multi-schema** — single schema per database
 - **DDL auto-commits** — migrations are not transactional (`transactional_ddl = False`)
 - **SQLAlchemy 2.0–2.1 only** — pinned to `<2.2` due to internal API dependencies ([details](docs/ARCHITECTURE.md))
-- **Async requires pycubrid >= 1.2.0,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
+- **Async requires pycubrid >= 1.3.2,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
 
 ## Documentation
 
@@ -228,8 +228,8 @@ async with AsyncSession(engine) as session:
 | CUBRID | 10.2, 11.0, 11.2, 11.4 |
 | SQLAlchemy | 2.0–2.1 |
 | Alembic | >=1.7 |
-| pycubrid (sync) | >=1.2.0,<2.0 |
-| pycubrid (async) | >=1.2.0,<2.0 |
+| pycubrid (sync) | >=1.3.2,<2.0 |
+| pycubrid (async) | >=1.3.2,<2.0 |
 
 ## FAQ
 
@@ -271,7 +271,7 @@ stmt = insert(users).values(name="Alice").on_duplicate_key_update(name="Alice Up
 
 ### Does sqlalchemy-cubrid support async?
 
-Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.2.0,<2.0`. All Core and ORM features work with `AsyncSession`.
+Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.3.2,<2.0`. Both pycubrid dialects use native `Connection.ping(False)` / `AsyncConnection.ping(False)` for `pool_pre_ping`, and all Core and ORM features work with `AsyncSession`.
 
 
 ## Related Projects

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/README.md

@@ -152,7 +152,7 @@ async with AsyncSession(engine) as session:
 - **No multi-schema** — single schema per database
 - **DDL auto-commits** — migrations are not transactional (`transactional_ddl = False`)
 - **SQLAlchemy 2.0–2.1 only** — pinned to `<2.2` due to internal API dependencies ([details](docs/ARCHITECTURE.md))
-- **Async requires pycubrid >= 1.2.0,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
+- **Async requires pycubrid >= 1.3.2,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
 
 ## Documentation
 
@@ -178,8 +178,8 @@ async with AsyncSession(engine) as session:
 | CUBRID | 10.2, 11.0, 11.2, 11.4 |
 | SQLAlchemy | 2.0–2.1 |
 | Alembic | >=1.7 |
-| pycubrid (sync) | >=1.2.0,<2.0 |
-| pycubrid (async) | >=1.2.0,<2.0 |
+| pycubrid (sync) | >=1.3.2,<2.0 |
+| pycubrid (async) | >=1.3.2,<2.0 |
 
 ## FAQ
 
@@ -221,7 +221,7 @@ stmt = insert(users).values(name="Alice").on_duplicate_key_update(name="Alice Up
 
 ### Does sqlalchemy-cubrid support async?
 
-Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.2.0,<2.0`. All Core and ORM features work with `AsyncSession`.
+Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.3.2,<2.0`. Both pycubrid dialects use native `Connection.ping(False)` / `AsyncConnection.ping(False)` for `pool_pre_ping`, and all Core and ORM features work with `AsyncSession`.
 
 
 ## Related Projects

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "sqlalchemy-cubrid"
-version = "1.4.1"
+version = "1.4.2"
 description = "CUBRID dialect for SQLAlchemy"
 readme = "README.md"
 license = "MIT"
@@ -54,7 +54,7 @@ alembic = [
     "alembic>=1.7",
 ]
 pycubrid = [
-    "pycubrid>=1.2.0,<2.0",
+    "pycubrid>=1.3.2,<2.0",
 ]
 
 [project.urls]

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/sqlalchemy_cubrid/__init__.py

@@ -52,7 +52,7 @@ from sqlalchemy.sql.sqltypes import (
     TIMESTAMP,
 )
 
-__version__ = "1.4.1"
+__version__ = "1.4.2"
 
 __all__ = (
     "insert",

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/sqlalchemy_cubrid/aio_pycubrid_dialect.py

@@ -7,6 +7,7 @@
 
 from __future__ import annotations
 
+from importlib import import_module
 from typing import Any, Callable, cast
 
 from sqlalchemy.connectors.asyncio import (
@@ -43,12 +44,15 @@ class AsyncAdapt_pycubrid_connection(AsyncAdapt_dbapi_connection):
     def autocommit(self, value: bool) -> None:
         self.await_(self._connection.set_autocommit(value))  # type: ignore[union-attr]
 
+    def ping(self, reconnect: bool = True) -> bool:
+        return self.await_(self._connection.ping(reconnect))  # type: ignore[union-attr]
+
 
 class AsyncAdapt_pycubrid_dbapi(AsyncAdapt_dbapi_module):
     def __init__(self, aio_module: Any) -> None:
         self._aio_module = aio_module
 
-        import pycubrid as sync_module
+        sync_module = import_module("pycubrid")
 
         self.paramstyle = sync_module.paramstyle
         self.Error = sync_module.Error
@@ -86,7 +90,7 @@ class PyCubridAsyncDialect(PyCubridDialect):
 
     @classmethod
     def import_dbapi(cls) -> DBAPIModule:
-        import pycubrid.aio as aio_module
+        aio_module = import_module("pycubrid.aio")
 
         return cast(DBAPIModule, AsyncAdapt_pycubrid_dbapi(aio_module))
 
@@ -118,13 +122,7 @@ class PyCubridAsyncDialect(PyCubridDialect):
         return connect
 
     def do_ping(self, dbapi_connection: Any) -> bool:
-        cursor = dbapi_connection.cursor()
-        try:
-            cursor.execute("SELECT 1")
-            cursor.fetchone()
-        finally:
-            cursor.close()
-        return True
+        return bool(dbapi_connection.ping(False))
 
 
 dialect = PyCubridAsyncDialect

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/sqlalchemy_cubrid/pycubrid_dialect.py

@@ -10,6 +10,7 @@
 from __future__ import annotations
 
 import logging
+from importlib import import_module
 from typing import Any, Callable, cast
 
 from sqlalchemy.engine.interfaces import DBAPIConnection, DBAPIModule, ConnectArgsType
@@ -70,7 +71,7 @@ class PyCubridDialect(CubridDialect):
     def import_dbapi(cls) -> DBAPIModule:
         """Import and return the pycubrid DBAPI module."""
         try:
-            import pycubrid as dbapi_module
+            dbapi_module = import_module("pycubrid")
         except ImportError as e:
             raise e
         log.debug("Loaded pycubrid DBAPI (version %s)", getattr(dbapi_module, "__version__", "?"))
@@ -125,18 +126,8 @@ class PyCubridDialect(CubridDialect):
         return connect
 
     def do_ping(self, dbapi_connection: DBAPIConnection) -> bool:
-        """Ping the server to check connection liveness.
-
-        pycubrid does not expose a native ``ping()`` method, so we
-        execute a lightweight query instead.
-        """
-        cursor = dbapi_connection.cursor()
-        try:
-            cursor.execute("SELECT 1")
-            cursor.fetchone()
-        finally:
-            cursor.close()
-        return True
+        """Ping using native pycubrid CHECK_CAS (FC=32). Requires pycubrid>=1.3.2."""
+        return bool(dbapi_connection.ping(False))
 
 
 dialect = PyCubridDialect

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2/sqlalchemy_cubrid.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sqlalchemy-cubrid
-Version: 1.4.1
+Version: 1.4.2
 Summary: CUBRID dialect for SQLAlchemy
 Author-email: Yeongseon Choe <yeongseon.choe@gmail.com>, Gyeongjun Paik <paikend@gmail.com>
 License-Expression: MIT
@@ -45,7 +45,7 @@ Requires-Dist: alembic>=1.7; extra == "dev"
 Provides-Extra: alembic
 Requires-Dist: alembic>=1.7; extra == "alembic"
 Provides-Extra: pycubrid
-Requires-Dist: pycubrid<2.0,>=1.2.0; extra == "pycubrid"
+Requires-Dist: pycubrid<2.0,>=1.3.2; extra == "pycubrid"
 Dynamic: license-file
 
 # sqlalchemy-cubrid
@@ -202,7 +202,7 @@ async with AsyncSession(engine) as session:
 - **No multi-schema** — single schema per database
 - **DDL auto-commits** — migrations are not transactional (`transactional_ddl = False`)
 - **SQLAlchemy 2.0–2.1 only** — pinned to `<2.2` due to internal API dependencies ([details](docs/ARCHITECTURE.md))
-- **Async requires pycubrid >= 1.2.0,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
+- **Async requires pycubrid >= 1.3.2,<2.0** — the `cubrid+aiopycubrid://` driver needs the async-capable pycubrid package line currently supported by this project
 
 ## Documentation
 
@@ -228,8 +228,8 @@ async with AsyncSession(engine) as session:
 | CUBRID | 10.2, 11.0, 11.2, 11.4 |
 | SQLAlchemy | 2.0–2.1 |
 | Alembic | >=1.7 |
-| pycubrid (sync) | >=1.2.0,<2.0 |
-| pycubrid (async) | >=1.2.0,<2.0 |
+| pycubrid (sync) | >=1.3.2,<2.0 |
+| pycubrid (async) | >=1.3.2,<2.0 |
 
 ## FAQ
 
@@ -271,7 +271,7 @@ stmt = insert(users).values(name="Alice").on_duplicate_key_update(name="Alice Up
 
 ### Does sqlalchemy-cubrid support async?
 
-Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.2.0,<2.0`. All Core and ORM features work with `AsyncSession`.
+Yes. Use `create_async_engine("cubrid+aiopycubrid://...")` with the pycubrid async driver. Requires `pycubrid>=1.3.2,<2.0`. Both pycubrid dialects use native `Connection.ping(False)` / `AsyncConnection.ping(False)` for `pool_pre_ping`, and all Core and ORM features work with `AsyncSession`.
 
 
 ## Related Projects

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/sqlalchemy_cubrid.egg-info/requires.txt

@@ -18,4 +18,4 @@ tox
 alembic>=1.7
 
 [pycubrid]
-pycubrid<2.0,>=1.2.0
+pycubrid<2.0,>=1.3.2

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/test/test_aio_pycubrid_dialect.py

@@ -2,6 +2,7 @@ from __future__ import annotations
 
 import sys
 import types
+from typing import Any, cast
 from unittest.mock import AsyncMock, MagicMock, patch
 
 from sqlalchemy.engine import url
@@ -43,7 +44,7 @@ class TestPyCubridAsyncDialectImportDbapi:
     def test_import_dbapi_returns_async_adapt_module(self):
         fake_aio = types.ModuleType("pycubrid.aio")
         fake_sync = types.ModuleType("pycubrid")
-        fake_sync.paramstyle = "qmark"  # type: ignore[attr-defined]
+        cast(Any, fake_sync).paramstyle = "qmark"
         for attr in [
             "Error",
             "OperationalError",
@@ -163,6 +164,20 @@ class TestAsyncAdaptPycubridConnection:
 
         assert isinstance(cur, AsyncAdapt_pycubrid_cursor)
 
+    def test_ping_awaits_underlying_async_ping(self):
+        mock_dbapi = MagicMock()
+        mock_async_conn = MagicMock()
+        mock_async_conn.ping.return_value = object()
+
+        conn = AsyncAdapt_pycubrid_connection(mock_dbapi, mock_async_conn)
+
+        with patch.object(conn, "await_", return_value=False) as mock_await:
+            result = conn.ping(False)
+
+        assert result is False
+        mock_async_conn.ping.assert_called_once_with(False)
+        mock_await.assert_called_once_with(mock_async_conn.ping.return_value)
+
 
 class TestAsyncAdaptPycubridCursor:
     def test_setinputsizes_is_noop(self):
@@ -192,15 +207,12 @@ class TestAsyncAdaptPycubridCursor:
 
 
 class TestPyCubridAsyncDialectDoPing:
-    def test_do_ping_executes_select_1(self):
+    def test_do_ping_propagates_boolean(self):
         dialect = PyCubridAsyncDialect()
         mock_conn = MagicMock()
-        mock_cursor = MagicMock()
-        mock_conn.cursor.return_value = mock_cursor
+        mock_conn.ping.return_value = False
 
         result = dialect.do_ping(mock_conn)
 
-        mock_cursor.execute.assert_called_once_with("SELECT 1")
-        mock_cursor.fetchone.assert_called_once()
-        mock_cursor.close.assert_called_once()
-        assert result is True
+        mock_conn.ping.assert_called_once_with(False)
+        assert result is False

{sqlalchemy_cubrid-1.4.1 → sqlalchemy_cubrid-1.4.2}/test/test_pycubrid_dialect.py

@@ -56,16 +56,10 @@ class TestPyCubridDialectBasics:
         assert imported is fake_module
 
     def test_import_dbapi_import_error(self):
-        import builtins
-
-        real_import = builtins.__import__
-
-        def _fake_import(name, *args, **kwargs):
-            if name == "pycubrid":
-                raise ImportError("pycubrid not installed")
-            return real_import(name, *args, **kwargs)
-
-        with patch("builtins.__import__", side_effect=_fake_import):
+        with patch(
+            "sqlalchemy_cubrid.pycubrid_dialect.import_module",
+            side_effect=ImportError("pycubrid not installed"),
+        ):
             with pytest.raises(ImportError, match="pycubrid not installed"):
                 PyCubridDialect.import_dbapi()
 
@@ -154,31 +148,25 @@ class TestPyCubridOnConnect:
 
 
 class TestPyCubridDoPing:
-    def test_do_ping_success(self):
+    def test_do_ping_propagates_true(self):
         dialect = PyCubridDialect()
         dbapi_conn = MagicMock()
-        cursor = MagicMock()
-        cursor.fetchone.return_value = (1,)
-        dbapi_conn.cursor.return_value = cursor
+        dbapi_conn.ping.return_value = True
 
         result = dialect.do_ping(dbapi_conn)
 
         assert result is True
-        cursor.execute.assert_called_once_with("SELECT 1")
-        cursor.fetchone.assert_called_once()
-        cursor.close.assert_called_once()
+        dbapi_conn.ping.assert_called_once_with(False)
 
-    def test_do_ping_always_closes_cursor(self):
+    def test_do_ping_propagates_false(self):
         dialect = PyCubridDialect()
         dbapi_conn = MagicMock()
-        cursor = MagicMock()
-        cursor.execute.side_effect = RuntimeError("connection lost")
-        dbapi_conn.cursor.return_value = cursor
+        dbapi_conn.ping.return_value = False
 
-        with pytest.raises(RuntimeError, match="connection lost"):
-            dialect.do_ping(dbapi_conn)
+        result = dialect.do_ping(dbapi_conn)
 
-        cursor.close.assert_called_once()
+        assert result is False
+        dbapi_conn.ping.assert_called_once_with(False)
 
 
 class TestPyCubridExecutionContext: