Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

Bugfix/Data inconsistency caused by updateEventHandler#3843

Merged
v1r3n merged 2 commits intoNetflix:mainfrom
wy-hua:bugfix/redis-dao-update-event-handler
Nov 12, 2023
Merged

Bugfix/Data inconsistency caused by updateEventHandler#3843
v1r3n merged 2 commits intoNetflix:mainfrom
wy-hua:bugfix/redis-dao-update-event-handler

Conversation

@wy-hua
Copy link
Contributor

@wy-hua wy-hua commented Nov 10, 2023

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

  1. If one event handler gets updated to listen on a new event, remove its previous index of event handlers by events

Describe the new behavior from this PR, and why it's needed
To resolve #3842

Alternatives considered

Describe alternative implementation you have considered

@wy-hua wy-hua changed the title Bugfix/redis dao update event handler Bugfix/Data inconsistency caused by updateEventHandler Nov 10, 2023
@v1r3n v1r3n merged commit be790a4 into Netflix:main Nov 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data inconsistency caused by updateEventHandler in RedisEventHandlerDAO.java

2 participants