Hi Chesnay, Thanks for the reply. I think it is reasonable to remove the configuration argument in AbstractUdfStreamOperator#open if it is consistently empty. I'll propose a discuss about the specific actions in FLINK-6912 at a later time.
Best, Wencong Liu At 2023-07-18 16:38:59, "Chesnay Schepler" <ches...@apache.org> wrote: >On 18/07/2023 10:33, Wencong Liu wrote: >> For FLINK-6912: >> >> There are three implementations of RichFunction that actually use >> the Configuration parameter in RichFunction#open: >> 1. ContinuousFileMonitoringFunction#open: It uses the configuration >> to configure the FileInputFormat. [1] >> 2. OutputFormatSinkFunction#open: It uses the configuration >> to configure the OutputFormat. [2] >> 3. InputFormatSourceFunction#open: It uses the configuration >> to configure the InputFormat. [3] > >And none of them should have any effect since the configuration is empty. > >See org.apache.flink.streaming.api.operators.AbstractUdfStreamOperator#open.