build(deps): sync pinned requirements for Dependabot pip PRs

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

pyproject.toml

@@ -42,7 +42,7 @@ dependencies = [
     # ``google-auth`` 2.53+ dropped it, so Superset must declare it
     # explicitly to keep fresh ``pip install apache-superset`` working
     # without the ``base.txt`` lock file (#40962).
-    "cachetools>=7.1.4, <8",
+    "cachetools>=7.1.6, <8",
     "celery>=5.6.3, <6.0.0",
     "click>=8.4.2",
     "click-option-group",

requirements/base.txt

@@ -46,7 +46,7 @@ cachelib==0.13.0
     # via
     #   flask-caching
     #   flask-session
-cachetools==7.1.4
+cachetools==7.1.6
     # via apache-superset (pyproject.toml)
 cattrs==25.1.1
     # via requests-cache

requirements/development.txt

@@ -101,7 +101,7 @@ cachelib==0.13.0
     #   -c requirements/base-constraint.txt
     #   flask-caching
     #   flask-session
-cachetools==7.1.4
+cachetools==7.1.6
     # via
     #   -c requirements/base-constraint.txt
     #   apache-superset

superset/sql_lab.py

@@ -161,9 +161,6 @@ def get_query(query_id: int) -> Query:
     try:
         return db.session.query(Query).filter_by(id=query_id).one()
     except Exception as ex:
-        # roll back so a poisoned session (e.g. PendingRollbackError after a
-        # failed flush) doesn't fail every subsequent backoff retry identically
-        db.session.rollback()
         raise SqlLabException("Failed at getting query") from ex
 
 

tests/unit_tests/sql_lab_test.py

@@ -35,7 +35,6 @@ from superset.sql.parse import SQLStatement, Table
 from superset.sql_lab import (
     execute_query,
     execute_sql_statements,
-    get_query,
     get_sql_results,
 )
 from superset.utils.rls import apply_rls, get_predicates_for_table
@@ -72,34 +71,6 @@ def test_execute_query(mocker: MockerFixture, app: None) -> None:
     SupersetResultSet.assert_called_with([(42,)], cursor.description, db_engine_spec)
 
 
-def test_get_query_rolls_back_session_before_retrying(
-    mocker: MockerFixture, app: SupersetApp
-) -> None:
-    """
-    A broken transaction (e.g. `PendingRollbackError` following a failed flush)
-    leaves the session unusable until `session.rollback()` is called, so without
-    it every `backoff` retry would reuse the same poisoned session and fail
-    identically. `get_query` must roll back on failure so each retry gets a
-    clean session and has a real chance to succeed.
-    """
-    # avoid actually sleeping through the `backoff` decorator's retry interval
-    mocker.patch("backoff._sync.time.sleep")
-
-    expected_query = mocker.MagicMock()
-    mock_one = mocker.patch("superset.sql_lab.db.session.query")
-    mock_one.return_value.filter_by.return_value.one.side_effect = [
-        Exception("session is broken"),
-        expected_query,
-    ]
-    mock_rollback = mocker.patch("superset.sql_lab.db.session.rollback")
-
-    result = get_query(query_id=1)
-
-    assert result is expected_query
-    assert mock_one.return_value.filter_by.return_value.one.call_count == 2
-    mock_rollback.assert_called_once()
-
-
 @with_config(
     {
         "SQLLAB_PAYLOAD_MAX_MB": 50,