krfb: fix termination and syncing of threads on exit #10

This resolves issue #2 and replaces PR #4.
I can't test functionality here, so this is purely a logical-based patch.
Please help testing it and let me know how it goes.

Unfortunately, this patch does not help solve the problem. Here is the error output - link.
Before these changes, the error output was like this - link.

ok, thanks for testing @ormorph. Will do more analysis on this and come back. Is there a simple way to test on a single machine? That would give me a way to reproduce the issue here.

Good news, I am able to reproduce the problem here, so I will now be able to test it myself.

Good news, I am able to reproduce the problem here, so I will now be able to test it myself.

This is good, I also tried it on one computer via virtualbox, it worked by creating a node adapter. Still, when you have a second computer with TDE, it is more convenient. But it is still possible to implement it.

I have published an updated patch. Tested here and I don't have crashes anymore. I also verified the threads terminate successfully in a controlled manner.
@ormorph can you test this patch again on your side?

have published an updated patch. Tested here and I don't have crashes anymore. I also verified the threads terminate successfully in a controlled manner.
@ormorph can you test this patch again on your side?

I didn't have any crashes, the patch works.

Thanks for testing @ormorph 🙂

Patch has been included in PSB and PTB repository for a long time for testing and no problems are reported. It seems to be time to merge it.

krfb: fix termination and syncing of threads on exit #10

Εξεταστές