Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

parser: Cleanup nginx-combined with splitFields() #22

Merged
merged 1 commit into from
Oct 2, 2024
Merged

parser: Cleanup nginx-combined with splitFields() #22

merged 1 commit into from
Oct 2, 2024

Conversation

iBug
Copy link
Collaborator

@iBug iBug commented Oct 2, 2024

No real performance difference:

Old impl:

real    0m16.944s
user    0m26.349s
sys     0m1.946s

New impl:

real    0m17.477s
user    0m26.257s
sys     0m2.149s

@iBug iBug force-pushed the nginx-combined-fields branch from cb5bdc4 to 423c06f Compare October 2, 2024 08:47
@iBug iBug merged commit 423c06f into master Oct 2, 2024
2 checks passed
@iBug iBug deleted the nginx-combined-fields branch October 2, 2024 08:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@taoky taoky taoky approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iBug @taoky