Skip to content

BI-2027 - Add More Detailed Logging Around Redis Cache#322

Merged
mlm483 merged 5 commits intodevelopfrom
feature/BI-2027
Dec 14, 2023
Merged

BI-2027 - Add More Detailed Logging Around Redis Cache#322
mlm483 merged 5 commits intodevelopfrom
feature/BI-2027

Conversation

@mlm483
Copy link
Contributor

@mlm483 mlm483 commented Dec 11, 2023

Description

Story: https://breedinginsight.atlassian.net/browse/BI-2027

  • Added cache key to all logs.
  • Added a debug log for every call to populate(UUID key).

Dependencies

None

Testing

None

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have tested that my code works with both the brapi-java-server and BreedBase
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <please include a link to TAF run>

@mlm483 mlm483 changed the title [BI-2027] - Add More Detailed Logging Around Redis Cache BI-2027 - Add More Detailed Logging Around Redis Cache Dec 11, 2023
@mlm483 mlm483 marked this pull request as ready for review December 11, 2023 20:15
@mlm483 mlm483 requested review from a team, dmeidlin and nickpalladino and removed request for a team December 11, 2023 20:15
@mlm483 mlm483 merged commit 5337008 into develop Dec 14, 2023
@mlm483 mlm483 deleted the feature/BI-2027 branch December 14, 2023 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dmeidlin dmeidlin dmeidlin approved these changes

@nickpalladino nickpalladino nickpalladino approved these changes

Assignees

@dmeidlin dmeidlin

Labels

feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mlm483 @dmeidlin @nickpalladino @rob-ouser-bi