xuyangzhong commented on PR #26648: URL: https://github.com/apache/flink/pull/26648#issuecomment-2957478386
Hi, @lincoln-lil thanks for review. This trait describes the inherent ability of the operator itself to output duplicate changes for operators downstream to consume. Each operator's trait is derived from its output, meaning that downstream operators determine whether this operator can output duplicate changes. `AllowDuplicateChanges` sounds more like a requirement imposed by the operator on upstream operators, but I want to express that it is a requirement derived from downstream components. How about "OutputDuplicateChanges"? > UNKNOWN seems redundant Ok, I'll remove this value. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
