[SERVER-79998] fold OpObserverImpl::onReplicationRollback() into rollback_impl.cpp Created: 15/Aug/23 Updated: 17/Aug/23 Resolved: 17/Aug/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Benety Goh | Assignee: | Backlog - Storage Execution Team |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Assigned Teams: |
Storage Execution
|
||||||||
| Participants: | |||||||||
| Description |
|
The OpObserver method onReplicationRollback() is implemented in one OpObserver - OpObserverImpl and has a single caller in RollbackImpl. It would simplify things to fold this OpObserver logic into the rollback code. |
| Comments |
| Comment by Benety Goh [ 17/Aug/23 ] |
|
OpObserverImpl is not the only OpObserver that implements this method. Last search for this method shows at least 12 other implementations. |