pnowojski commented on a change in pull request #11403: [FLINK-16316][operators] Implement new StreamOperatorBase as a replacement for AbstractStreamOperator URL: https://github.com/apache/flink/pull/11403#discussion_r397100621
########## File path: flink-streaming-java/src/main/java/org/apache/flink/streaming/api/operators/YieldingOperatorFactory.java ########## @@ -17,10 +17,13 @@ package org.apache.flink.streaming.api.operators; +import org.apache.flink.annotation.PublicEvolving; + /** * An operator that needs access to the {@link MailboxExecutor} to yield to downstream operators needs to be created * through a factory implementing this interface. */ +@PublicEvolving public interface YieldingOperatorFactory<OUT> extends StreamOperatorFactory<OUT> { Review comment: Maybe let's keep it for now as it is? As I'm not sure if I like the idea of nullifying parameters based on `needsMailboxExecutor()`. I think I would prefer mixed pattern to that. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services