Skip to content

Conversation

@euri10
Copy link
Collaborator

@euri10 euri10 commented Nov 21, 2025

Description

Closes

@euri10
Copy link
Collaborator Author

euri10 commented Nov 21, 2025

it still fails randomly between this commit and the last one, there is no reason one works and the other not as he change is just a typo

FAILED docs/examples/usage/usage_drivers_and_querying_10.py::test_example_10_duckdb_config - _duckdb.IOException: IO Error: Could not set lock on file "/home/runner/work/sqlspec/sqlspec/analytics.duckdb": Conflicting lock is held in /home/runner/.local/share/uv/python/cpython-3.10.19-linux-x86_64-gnu/bin/python3.10 (PID 2210). See also https://duckdb.org/docs/stable/connect/concurrency

i guess it is whether or not pytest xdist picks on the same / or not same process, but I thought using at tmp_path would mean different files but i might be incorrect on that ,

any idea @cofin I see marks in tests maybe we can mark duckdb tests to run on the same process ?

@euri10
Copy link
Collaborator Author

euri10 commented Nov 21, 2025

ok seems like it is good now

@euri10 euri10 changed the title fix: use tmp_path to avoid duckdb locks in testing docs fix: avoid duckdb locks in testing docs Nov 21, 2025
@cofin cofin merged commit 93fbb4b into litestar-org:main Nov 21, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants