afedulov commented on PR #20757: URL: https://github.com/apache/flink/pull/20757#issuecomment-1277604779
>Looks like the DataGenTableSource may use the stateful SequenceGenerator (DataGenTableSourceFactory#createContainer)?. Oh, indeed, missed that one. However, it looks like the stateful property of the generated sequences is already covered by the underlying `NumberSequenceSource` that we use internally, so it should be a matter of simply using a deterministic `GeneratorFunction` to achieve the same results (i.e mapping Long to Integer, Short, Byte etc.). > I would deprecated it now so we have some incentive to migrate existing usages (in particular internal!) to migrate to the new source. Marked as deprecated, but I am not sure if simple "Use [fullpath]DataGeneratorSource instead." is enough here. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
