Re-use logger instance (#2029) #2031

vikasmb · 2022-07-12T23:07:06Z

Re-use logger instance

Existing logger initialization constructed different logger
instances upon call to Get() method.
Fixed the initialization logic to re-use the logger instance.

Added unit tests for logger initialization fix

Cherry-pick of PR #2029

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

* Re-use logger instance - Existing logger initialization constructed different logger instances upon call to Get() method. - Fixed the initailiation logic to re-use the logger instance. * Added unit tests for logger initialization fix

Re-use logger instance (#2029)

99e72e7

* Re-use logger instance - Existing logger initialization constructed different logger instances upon call to Get() method. - Fixed the initailiation logic to re-use the logger instance. * Added unit tests for logger initialization fix

vikasmb requested review from orsenthil, jayanthvn and achevuru July 12, 2022 23:07

vikasmb requested a review from a team as a code owner July 12, 2022 23:07

orsenthil approved these changes Jul 12, 2022

View reviewed changes

orsenthil merged commit cbd10b2 into aws:release-1.11 Jul 12, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Re-use logger instance (#2029) #2031

Re-use logger instance (#2029) #2031

vikasmb commented Jul 12, 2022 •

edited

Loading

Re-use logger instance (#2029) #2031

Re-use logger instance (#2029) #2031

Conversation

vikasmb commented Jul 12, 2022 • edited Loading

vikasmb commented Jul 12, 2022 •

edited

Loading