Skip to content

Commit 7056f74

Browse files
committed
Update adr-023-event-driven-sync.md
1 parent aa1b5a2 commit 7056f74

1 file changed

Lines changed: 0 additions & 12 deletions

File tree

docs/adr/adr-023-event-driven-sync.md

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -333,23 +333,11 @@ Accepted and Implemented
333333
- **Efficiency**: No CPU cycles wasted on polling when idle
334334
- **Lower Latency**: Immediate reaction to new data (no polling interval delay)
335335
- **Cleaner Code**: Separation of concerns makes code easier to understand
336-
- **Better Testability**: Components can be tested in isolation
337-
- **Extensibility**: Easy to add new event types or subscribers
338-
- **Resource Optimization**: Better CPU and memory usage patterns
339336

340337
### Negative
341338

342-
- **Additional Component**: Notifier adds another layer to understand
343-
- **Event Buffer Management**: Need to configure channel buffer sizes appropriately
344-
- **Debugging Complexity**: Event flow can be harder to trace than direct calls
345-
- **Memory Overhead**: Each subscription maintains a buffered channel (minimal but measurable)
346-
347339
### Neutral
348340

349-
- **Learning Curve**: Developers need to understand event-driven patterns
350-
- **Event Semantics**: Need clear documentation on what events mean and when they're published
351-
- **Order Guarantees**: Events are delivered in order per subscription but no cross-subscription ordering
352-
- **Backpressure**: Slow consumers could fill buffers, but this is intentional design
353341

354342
## References
355343

0 commit comments

Comments
 (0)