SteNicholas commented on pull request #13637: URL: https://github.com/apache/flink/pull/13637#issuecomment-717201338
> @SteNicholas we shouldn't change the argument order because that would break existing code. That's why now the new `fromSequence()` doesn't have anything besides `from` and `to`. To keep it to a minimum. > > Thanks for the comment on [411b4c1](https://github.com/apache/flink/commit/411b4c1e4ad88a7468343efa04dd87c12d0dc5f8), I'm fixing it. 👌 > > I talked to Stephan myself, he's also fine with the changes. @aljoscha OK, I have no problem. Go ahead to merge this pull request. ---------------------------------------------------------------- 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]
