Github user fhueske commented on a diff in the pull request:
https://github.com/apache/flink/pull/4813#discussion_r144268057
--- Diff:
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/batch/table/validation/SortValidationTest.scala
---
@@ -27,10 +27,26 @@ import org.junit._
class SortValidationTest extends TableTestBase {
--- End diff --
Right, I'll add a test that validates that
`table.orderBy('a.asc).offset(10).offset(10)` is rejected.
`table.orderBy('a.asc).fetch(5)` is already supported as you suggested (see
the modified `SortITCase`).
---
