You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "raboof (via GitHub)" <gi...@apache.org> on 2024/07/29 15:18:03 UTC
[I] Deprecate `org.apache.pekko.dispatch.Futures` [pekko]
raboof opened a new issue, #1417:
URL: https://github.com/apache/pekko/issues/1417
I think this class was introduced when Akka still supported Java versions before Java 8, which introduced `CompletionStage`.
We should probably try and deprecate `org.apache.pekko.dispatch.Futures` and any Java APIs that still produce/expect `Future`s, and introduce ones that work with `CompletionStage`.
--
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: notifications-unsubscribe@pekko.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
Re: [I] Deprecate `org.apache.pekko.dispatch.Futures` [pekko]
Posted by "raboof (via GitHub)" <gi...@apache.org>.
raboof commented on issue #1417:
URL: https://github.com/apache/pekko/issues/1417#issuecomment-2257026633
I didn't intend to suggest Java users should use `Future$.MODULE$.failed` and such: what I was thinking is that Java users shouldn't need to touch the Scala `Future` at all, and we should make sure that where they currently must use `Future`, we provide equivalent APIs using the more Java-native `CompletionStage`.
Something along the lines of (sketch) https://github.com/apache/pekko/compare/main...raboof:pekko:no-scala-futures-in-java?expand=1
--
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: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
Re: [I] Deprecate `org.apache.pekko.dispatch.Futures` [pekko]
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on issue #1417:
URL: https://github.com/apache/pekko/issues/1417#issuecomment-2256710534
I had a quick look at this and I think this Futures class is useful. See https://github.com/apache/pekko/pull/1418 for the PR that convinced that this change is not a good idea.
--
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: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
Re: [I] Deprecate `org.apache.pekko.dispatch.Futures` [pekko]
Posted by "pjfanning (via GitHub)" <gi...@apache.org>.
pjfanning commented on issue #1417:
URL: https://github.com/apache/pekko/issues/1417#issuecomment-2257037007
90% of the internal Pekko usages of org.apache.pekko.dispatch.Futures are in Java unit tests.
I guess we could move the code to a test class and deprecate the org.apache.pekko.dispatch.Futures class.
Some of the tests are linked in the Paradox documentation though.
FutureTimeoutSupport has one usage that could moved internal to that class.
--
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: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org
Re: [I] Deprecate `org.apache.pekko.dispatch.Futures` [pekko]
Posted by "He-Pin (via GitHub)" <gi...@apache.org>.
He-Pin closed issue #1417: Deprecate `org.apache.pekko.dispatch.Futures`
URL: https://github.com/apache/pekko/issues/1417
--
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: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org