GitHub user Ben-Zvi opened a pull request: https://github.com/apache/drill/pull/1219
DRILL-6339: Add a new option to disable TopN (for testing) To allow simple testing of the External Sort operator (e.g., test spilling), we would like to use a query of the form: ".... ORDER BY ... LIMIT ... "; however the current planner unconditionally selects the TopN operator for such a query. Solution: Introduce an internal option that can negate the planner's choice in this case. The implementation overrides the matches() method (default was returning true) in PushLimitToTopN , returning the new boolean option (default true) instead. (The last 'true' in SystemOptionManager makes this option 'internal'). This new option was tested manually. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Ben-Zvi/drill DRILL-6339 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/drill/pull/1219.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1219 ---- commit 0c5bb6be7d62039dbd65e5fb06f949178bcd36eb Author: Ben-Zvi <bben-zvi@...> Date: 2018-04-18T00:05:50Z DRILL-6339: Add a new option to disable TopN (for testing) ---- ---