Skip to content

Update test_lsh_cassandra.py#139

Open
researcher2 wants to merge 5 commits intoekzhu:masterfrom
researcher2:master
Open

Update test_lsh_cassandra.py#139
researcher2 wants to merge 5 commits intoekzhu:masterfrom
researcher2:master

Conversation

@researcher2
Copy link

Add test case to demonstrate hanging when unpickling lsh in multiple processes.

Add test case to demonstrate hanging when unpickling lsh in multiple processes.
@researcher2
Copy link
Author

Hey, test case to demonstrate issue with #135. Please make whatever changes required for project coding standards.

@ekzhu
Copy link
Owner

ekzhu commented Nov 1, 2020

Hey I made some changes to the pull request so the pickled lsh object is directly passed into the forked processes. Does this work for you? There is still an assertion failure due to time.

@researcher2
Copy link
Author

researcher2 commented Nov 15, 2020

Hello, unfortunately passing directly didn't work for me either. The only way I could get it to work some of the time was by creating a new object from scratch in each process with the same basename.

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