mdedetrich commented on issue #2178: URL: https://github.com/apache/pekko/issues/2178#issuecomment-3266796907
> holder.elem match { > case Success(elem) => > if (elem != null) { > push(out, elem) > pullIfNeeded() > } else { > // elem is null > pullIfNeeded() > pushNextIfPossible() > } > You can see, in mapAsync , when the result value is `null`, it just drop it and `pull` upstream again, does that wrong? here is the same, but more explicitly with `mapNotNull/collectNotNull` If it does that, its either illegal (see https://github.com/reactive-streams/reactive-streams-jvm#2.13 and hence not following the reactive/java streams specification) or its an internal optimization detail thats not publicly visible to users. -- 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