Skip to content

Conversation

Tishj
Copy link
Contributor

@Tishj Tishj commented Dec 5, 2022

This PR actually fixes #5520

#5553 was a bandaid fix, as suspected

@hannes hannes added this to the 0.6.1 milestone Dec 5, 2022
@Mytherin Mytherin removed this from the 0.6.1 milestone Dec 5, 2022
@Mytherin
Copy link
Collaborator

Mytherin commented Dec 5, 2022

Looks like there is still an issue with 32-bit systems - let’s postpone this until after the bugfix release.

@Tishj
Copy link
Contributor Author

Tishj commented Dec 5, 2022

Yea I was looking at the issue just now, probably gonna need to run this on Linux on AWS

@Tishj
Copy link
Contributor Author

Tishj commented Dec 7, 2022

Made a linux docker container to test this, but..

Filters: test/sql/catalog/table/test_table_drop_concurrent.test
[1/1] (100%): test/sql/catalog/table/test_table_drop_concurrent.test            
===============================================================================
All tests passed (8000 assertions in 1 test case)

it passes.., even in a loop
so it seems the issue is related to 32 bit, not Linux

EDIT: On AWS I can reproduce it!
Only problem: that was a release build, building debug now

@Tishj
Copy link
Contributor Author

Tishj commented Dec 8, 2022

codecov failed to upload again

@Mytherin Mytherin merged commit 0f2becd into duckdb:master Dec 8, 2022
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.

[Python] Segfault in DuckDBPyConnection.unregister() in multi-threaded program
3 participants