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

Switched the internal plugin logger from fmt to zerolog + Added optional parameter of ydb output plugin LogLevel #14

Merged
merged 1 commit into from
May 3, 2024

Conversation

asmyasnikov
Copy link
Member

@asmyasnikov asmyasnikov commented May 3, 2024

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: #13

What is the new behavior?

Other information

@asmyasnikov asmyasnikov self-assigned this May 3, 2024
@asmyasnikov asmyasnikov changed the title Switched the internal plugin logger from fmt to zerolog with default logging level zerolog.Disabled + Added optional parameter of ydb output plugin LogLevel (if not defined - plugin logging disabled) Switched the internal plugin logger from fmt to zerolog + Added optional parameter of ydb output plugin LogLevel May 3, 2024
@asmyasnikov asmyasnikov force-pushed the zerolog branch 3 times, most recently from b83f195 to a5f9155 Compare May 3, 2024 08:46
…fault logging level `zerolog.Disabled`

* Added optional parameter of ydb output plugin `LogLevel` (if not defined - plugin logging disabled)
@asmyasnikov asmyasnikov merged commit 429fbba into main May 3, 2024
6 checks passed
@asmyasnikov asmyasnikov deleted the zerolog branch May 3, 2024 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant