pjfanning commented on code in PR #2001:
URL: https://github.com/apache/pekko/pull/2001#discussion_r2249961392


##########
actor/src/main/scala/org/apache/pekko/pattern/Patterns.scala:
##########
@@ -126,7 +126,8 @@ object Patterns {
    *   timeout);
    * }}}
    */
-  def askWithReplyTo(actor: ActorRef, messageFactory: japi.Function[ActorRef, 
Any], timeout: Timeout): Future[AnyRef] =
+  def askWithReplyTo(actor: ActorRef, messageFactory: 
japi.function.Function[ActorRef, Any], timeout: Timeout)

Review Comment:
   I'll go with the consensus on this but it would be useful if we knew if 
external users will need to rewrite their code that uses our Java stream API 
due to this change or if a recompile is enough. I'm making assumptions but if 
Java use lambdas already when passing functions to our APIs then a recompile 
should be enough. But if they have code that uses japi.Function directly (or 
any of the other function classes that are being removed) then the user needs 
to do a code rewrite.
   



-- 
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-unsubscr...@pekko.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org
For additional commands, e-mail: notifications-h...@pekko.apache.org

Reply via email to