vvysotskyi commented on a change in pull request #1353: DRILL-6553: Fix TopN 
for unnest operator
URL: https://github.com/apache/drill/pull/1353#discussion_r199933855
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/planner/common/DrillLateralJoinRelBase.java
 ##########
 @@ -41,24 +41,22 @@
 
 
 public abstract class DrillLateralJoinRelBase extends Correlate implements 
DrillRelNode {
-
-  final private static double CORRELATE_MEM_COPY_COST = 
DrillCostBase.MEMORY_TO_CPU_RATIO * DrillCostBase.BASE_CPU_COST;
-  final public boolean excludeCorrelateColumn;
-  public DrillLateralJoinRelBase(RelOptCluster cluster, RelTraitSet traits, 
RelNode left, RelNode right, boolean excludeCorrelateCol,
-                               CorrelationId correlationId, ImmutableBitSet 
requiredColumns, SemiJoinType semiJoinType) {
+final private static double CORRELATE_MEM_COPY_COST = 
DrillCostBase.MEMORY_TO_CPU_RATIO * DrillCostBase.BASE_CPU_COST;
+  final public boolean excludeCorrelateColumn;  public 
DrillLateralJoinRelBase(RelOptCluster cluster, RelTraitSet traits, RelNode 
left, RelNode right,boolean excludeCorrelateCol,
 
 Review comment:
   Sorry, for some reason these lines were moved after the rebase 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to