[SPARK-13254][SQL] Fix planning of TakeOrderedAndProject operator

[JoshRosen]

The patch for SPARK-8964 ("use Exchange to perform shuffle in Limit" / #7334) inadvertently broke the planning of the TakeOrderedAndProject operator: because ReturnAnswer was the new root of the query plan, the TakeOrderedAndProject rule was unable to match before BasicOperators.

This patch fixes this by moving the TakeOrderedAndCollect and CollectLimit rules into the same strategy.

In addition, I made changes to the TakeOrderedAndProject operator in order to make its doExecute() method lazy and added a new TakeOrderedAndProjectSuite which tests the new code path.

/cc @davies and @marmbrus for review.

[davies]

LGTM, pending tests.

[SparkQA]

Test build #51008 has finished for PR 11145 at commit 2ccf9ed.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #51013 has finished for PR 11145 at commit 5cd4aa4.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[JoshRosen]

Alright, merging this to master.