twalthr commented on pull request #13298: URL: https://github.com/apache/flink/pull/13298#issuecomment-694883904
> limit(3).limit(10) doesn't make sense and equals to limit(3) Currently, it equals to `limit(10)`. The last limit overrides the previous limit. This can be confusing. A limit should only be set once. After that a new sort operation should be created. I agree that it doesn't make much sense but we should not introduce pre-optimization 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. For queries about this service, please contact Infrastructure at: [email protected]
