Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP Make HtmlRewriter Send + Sync. #217

Closed
wants to merge 1 commit into from

Conversation

orium
Copy link
Member

@orium orium commented Aug 6, 2024

No description provided.

@orium orium force-pushed the html-rewriter-send-sync branch from b68608a to 61d33f9 Compare August 9, 2024 11:47
@orium orium force-pushed the html-rewriter-send-sync branch from 61d33f9 to 8386a1a Compare August 9, 2024 11:54
@orium
Copy link
Member Author

orium commented Aug 12, 2024

The performance of the rewriter is about 60% slower if we make it Send + Sync. For that reason I'll drop the PR.

@orium orium closed this Aug 12, 2024
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.

1 participant