He-Pin opened a new pull request, #3383:
URL: https://github.com/apache/pekko/pull/3383

   ### Motivation
   Issue #2168 requests removal of stale FIXME/TODO comments. The dispatch 
module contained three comments referencing Akka 2.3/2.4 migration periods that 
are long over.
   
   ### Modification
   - Remove the `BalancingDispatcher` shortcut case in `Dispatchers` that threw 
a deprecation migration error (`FIXME remove in Akka 2.4`). Users can still use 
`BalancingDispatcherConfigurator` via full class name.
   - Remove the `mailbox-size` warning in `Mailboxes` (`TODO remove in Akka 
2.3`) along with its associated state variable.
   - Remove the stale `TODO RK remove these two for Akka 2.3` comment on 
`unbounded`/`bounded` mailbox shortcuts which are still actively used in 
`reference.conf`.
   
   ### Result
   Cleaner codebase without misleading stale migration comments. No behavioral 
change for supported configurations.
   
   ### Tests
   - `sbt "actor-tests/Test/testOnly 
org.apache.pekko.actor.dispatch.DispatchersSpec"` — 22 tests passed
   - `sbt "actor-tests/Test/testOnly 
org.apache.pekko.actor.dispatch.BalancingDispatcherSpec 
org.apache.pekko.actor.ActorMailboxSpec"` — 34 tests passed
   
   ### References
   Refs #2168


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to