Skip to content

Attempt to make D:OD fork safe#27

Closed
aklaswad wants to merge 2 commits intosixapart:masterfrom
Peatix:fork-safe
Closed

Attempt to make D:OD fork safe#27
aklaswad wants to merge 2 commits intosixapart:masterfrom
Peatix:fork-safe

Conversation

@aklaswad
Copy link

This commit add these functions to D:OD, when forked;

  • Reset all transaction status in child process
  • Destroy and forget all cached DBI handles safely

notes:

  • Works only when POSIX::AtFork was found
  • Basic logic was taken from Cache::Memcached::Fast::Safe, and some are
    taken from Teng/DBIx::Skinny

Disclaimer:

This commit add these functions to D:OD, when forked;

 - Reset all transaction status in child process
 - Destroy and forget all cached DBI handles safely

notes:
 - Works only when POSIX::AtFork was found
 - Basic logic was taken from Cache::Memcached::Fast::Safe, and some are
 taken from Teng/DBIx::Skinny
@coveralls
Copy link

coveralls commented Mar 19, 2019

Coverage Status

Coverage decreased (-0.3%) to 83.41% when pulling 42e1f86 on Peatix:fork-safe into 42d1546 on sixapart:master.

This was referenced Oct 26, 2020
@aklaswad
Copy link
Author

Thanks for better fix!

@aklaswad aklaswad closed this Mar 13, 2021
@aklaswad aklaswad deleted the fork-safe branch March 14, 2021 00:11
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