This is an automated email from the ASF dual-hosted git repository.
sadpandajoe pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new 472c3e4afd8 fix(tests): make deep nested sidecar fallback test
interpreter-independent (#45657)
472c3e4afd8 is described below
commit 472c3e4afd8b10b2734b2588a19378b0bbcf72f3
Author: Joe Li <[email protected]>
AuthorDate: Fri Oct 23 13:49:53 2026 -0700
fix(tests): make deep nested sidecar fallback test interpreter-independent
(#45657)
Co-authored-by: Claude Opus 5.5 <[email protected]>
---
tests/unit_tests/sql/execution/test_async_results.py | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/tests/unit_tests/sql/execution/test_async_results.py
b/tests/unit_tests/sql/execution/test_async_results.py
index 40e66ccb4ab..bcc69ed9a38 100644
--- a/tests/unit_tests/sql/execution/test_async_results.py
+++ b/tests/unit_tests/sql/execution/test_async_results.py
@@ -1328,12 +1328,19 @@ def test_deep_nested_sidecar_falls_back_to_arrow(
for _ in range(550):
value = [value]
result_set = _result_set(["value"], [(value,)])
+ # The depth at which encoding exceeds the recursion limit varies with the
+ # interpreter (Python 3.12 inlines comprehensions), so raise it explicitly.
+ encode = mocker.patch(
+ "superset.sql.execution.serialization._encode_nested_value",
+ side_effect=RecursionError,
+ )
result = _store_and_read(
mocker,
"SELECT value",
[("SELECT value", result_set, 1.0, 1)],
write_msgpack=use_msgpack,
)
+ encode.assert_called_once()
assert result.status == QueryStatus.SUCCESS, result.error_message
pd.testing.assert_frame_equal(
result.statements[0].data,
