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

[1_22] upgrade lolly to 1.4.27 #38

Merged
merged 1 commit into from
Jul 19, 2024
Merged

Conversation

jingkaimori
Copy link
Contributor

@jingkaimori jingkaimori commented Jul 18, 2024

Performance

Before

ns/character character/s err% total benchmark
26.92 37,150,771.29 2.8% 0.04 parsing large group of simple element
31.18 32,073,540.99 1.9% 0.05 serializing large group of simple element
48.78 20,499,900.93 2.5% 0.07 parsing group of complex tree
55.67 17,962,913.36 1.8% 0.08 parsing single tree with complex structure
34.32 29,139,808.31 1.5% 0.05 serializing group of complex tree
32.57 30,706,217.56 1.6% 0.05 serializing single tree with complex structure

After

ns/character character/s err% total benchmark
26.88 37,204,964.49 2.0% 0.04 parsing large group of simple element
28.82 34,699,794.81 0.6% 0.05 serializing large group of simple element
48.51 20,614,859.09 0.9% 0.07 parsing group of complex tree
52.53 19,035,944.20 1.0% 0.08 parsing single tree with complex structure
33.56 29,799,403.35 1.4% 0.05 serializing group of complex tree
30.82 32,449,871.18 2.7% 0.05 serializing single tree with complex structure

@jingkaimori jingkaimori requested a review from da-liii July 18, 2024 04:17
@da-liii da-liii changed the title upgrade lolly to 1.4.27 [1_22] upgrade lolly to 1.4.27 Jul 19, 2024
Copy link
Contributor

@da-liii da-liii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@da-liii da-liii merged commit 0f5205f into main Jul 19, 2024
3 checks passed
@da-liii da-liii deleted the jk/unnumbered/lolly-upgrade branch July 19, 2024 22:28
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