Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Drill >> mail # dev >> Review Request 14027: Adding Limit operator end to end


Copy link to this message
-
Review Request 14027: Adding Limit operator end to end

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14027/
-----------------------------------------------------------

Review request for drill and Jacques Nadeau.
Repository: drill-git
Description
-------

Adding Limit operator end to end
Diffs
-----

  common/src/main/java/org/apache/drill/common/logical/data/Limit.java 1774790
  exec/java-exec/src/main/java/org/apache/drill/exec/opt/BasicOptimizer.java c116b59
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractPhysicalVisitor.java c997db4
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/PhysicalVisitor.java 97e6795
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/Limit.java PRE-CREATION
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ImplCreator.java 9984454
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/limit/LimitBatchCreator.java PRE-CREATION
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/limit/LimitRecordBatch.java PRE-CREATION
  exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/StatsCollector.java 2ef5295
  exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/limit/TestSimpleLimit.java PRE-CREATION
  exec/java-exec/src/test/resources/limit/test1.json PRE-CREATION
  sqlparser/pom.xml 84b17a0
  sqlparser/src/main/java/org/apache/drill/optiq/DrillLimitRel.java PRE-CREATION
  sqlparser/src/main/java/org/apache/drill/optiq/DrillLimitRule.java PRE-CREATION
  sqlparser/src/main/java/org/apache/drill/optiq/DrillOptiq.java e687435
  sqlparser/src/main/java/org/apache/drill/optiq/DrillSortRel.java 64995c5
  sqlparser/src/main/java/org/apache/drill/optiq/DrillSortRule.java 0d9852a

Diff: https://reviews.apache.org/r/14027/diff/
Testing
-------
Thanks,

Timothy Chen