You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix: Permission Error trying to use pb.load_data_set() with duckdb on Windows #244
This is a Windows-specific issue with DuckDB file locking and temporary file cleanup. The error occurs because:
File Handle Not Released: When pb.load_dataset() creates a DuckDB database file in a temporary directory, the database connection isn't being properly closed before Python tries to clean up the temporary directory.
Windows File Locking: Windows is more strict about file locking than Unix systems - it won't allow deletion of files that have open handles, even if the process that opened them is trying to delete them.
Hmm, this is my bad for not testing this on Windows. I'm glad you did identify this issue though; maybe there should be some Windows-specific disconnect statements applied in key places (thanks for starting the dedicated PR). Also, I guess we ought to expand the CI testing to include Windows tests?
Yeah that sounds like a reasonable alternative. I'll look into adding some logic to handle Windows differently. I think Windows tests for CI would make sense too.
Activity
rich-iannone commentedon Jul 17, 2025
Hmm, this is my bad for not testing this on Windows. I'm glad you did identify this issue though; maybe there should be some Windows-specific disconnect statements applied in key places (thanks for starting the dedicated PR). Also, I guess we ought to expand the CI testing to include Windows tests?
matt-humphrey commentedon Jul 21, 2025
Yeah that sounds like a reasonable alternative. I'll look into adding some logic to handle Windows differently. I think Windows tests for CI would make sense too.
rich-iannone commentedon Oct 21, 2026
Fixed in 9241a4c. The table is now copied into in-memory DuckDB (
validate.py:707-733).