It seems confusing that there's a class named MergingWindowFnRunner that extends WindowMergingFnRunner. Maybe name it something like WindowMergingFnRunnerImpl? Or if you want to be descriptive, maybe MergingViaWindowFnRunner, or WindowMergingViaWindowFnRunner, or some other variation like that.
[ Full content available at: https://github.com/apache/beam/pull/6222 ] This message was relayed via gitbox.apache.org for [email protected]
