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.

Sign up for GitHub

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

Jump to bottom

Fieldmapper improvements #57

Merged

osiegmar merged 2 commits into osiegmar:master from nemecec:fieldmapper-improvements

Jan 18, 2021

nemecec commented Jan 17, 2021

Two small improvement suggestions (related to issue #55):

Include the actual pattern constant in the exception message, to make sure they do not get out of sync
Avoid map lookup for better performance


          Include the actual pattern constant in the exception message, to make…

9e9f690

… sure they do not get out of sync

nemecec mentioned this pull request

Ability to add custom fields to GelfMessage, computed from ILoggingEvent #55

Closed


          Avoid map lookup for better performance

14d7178

nemecec force-pushed the fieldmapper-improvements branch from a9c2830 to 14d7178 Compare

January 17, 2021 21:23

osiegmar merged commit bf8180a into osiegmar:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet