StephanEwen commented on pull request #13637:
URL: https://github.com/apache/flink/pull/13637#issuecomment-717084535
Okay, lets add the name into the method. So basically have
`fromSequence("name", from to)`. That sounds good.
@aljoscha and @SteNicholas the one worry I have about not adding a variant
with watermark strategy is that it cannot be used at all for streaming use
cases, because in the future we don't have the case any more where watermarks
are generated later in the stream.
Maybe let's think how a `fromElements()` method would look like. Would it
have a WatermarkStrategy or not. I think that `fromSequence()` and
`fromElemets()` should be symmetric there.
----------------------------------------------------------------
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:
[email protected]
