Skip to content

Add process forking warning to Python Core SDK documentation#3184

Merged
tore-statsig merged 3 commits intomainfrom
devin/1751581546-python-fork-warning
Jul 3, 2025
Merged

Add process forking warning to Python Core SDK documentation#3184
tore-statsig merged 3 commits intomainfrom
devin/1751581546-python-fork-warning

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Jul 3, 2025

No description provided.

- Add comprehensive warning about not forking processes after Statsig initialization
- Include technical explanation of threading/async runtime issues
- Provide correct examples for uWSGI and Gunicorn deployment scenarios
- Show incorrect initialization pattern to avoid
- Offer lazy initialization as alternative solution
- Addresses potential deadlock scenarios in production WSGI deployments

Co-Authored-By: xin@statsig.com <xin@statsig.com>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@jasonwzm
Copy link
Member

jasonwzm commented Jul 3, 2025

🚀 Deployed on https://686712dc51106e5d6ea0228c--cozy-fox-0defec.netlify.app

@jasonwzm jasonwzm temporarily deployed to pull request July 3, 2025 22:34 Inactive
- Resolves spellcheck CI failure for Python Core SDK documentation
- Gunicorn is a legitimate Python WSGI HTTP Server name used in examples

Co-Authored-By: xin@statsig.com <xin@statsig.com>
@jasonwzm jasonwzm temporarily deployed to pull request July 3, 2025 22:39 Inactive
@tore-statsig tore-statsig merged commit ce5ec87 into main Jul 3, 2025
2 checks passed
@tore-statsig tore-statsig deleted the devin/1751581546-python-fork-warning branch July 3, 2025 23:29
@jasonwzm jasonwzm temporarily deployed to pull request July 3, 2025 23:32 Inactive
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