SteNicholas commented on pull request #13676: URL: https://github.com/apache/flink/pull/13676#issuecomment-716507003
> @SteNicholas Thanks for your work! I pushed some changes to your branch, I hope that's ok. I split the unrelated changes that remove imports. I also fixed the `PatternStreamBuilder`, the problem with the code before was that the time characteristic would be dropped when calling the builder methods that don't call the right constructor. And I changed `DataStreamMatch` to use an explicit type instead of a boolean, because it's hard to parse what they mean when looking at the code in passing. > > Please let me know if you're ok with the changed, then I'd merge once CI is green. @aljoscha Your changes are ok for me. Please go ahead to do some changes. And no any comments with my changes? ---------------------------------------------------------------- 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]
