[GitHub] tajo pull request: Tajo 830: Some filter conditions with a SUBQUER...

[hyunsik]

Github user hyunsik commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/12#discussion_r13029221
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/engine/planner/rewrite/FilterPushDownRule.java
 ---
    @@ -232,50 +298,84 @@ public LogicalNode visitJoin(Set<EvalNode> cnf, 
LogicalPlan plan, LogicalPlan.Qu
           if (joinNode.getJoinType() == JoinType.CROSS) {
             joinNode.setJoinType(JoinType.INNER);
           }
    -      cnf.removeAll(matched);
    +      context.workingEvals.removeAll(matched);
         }
     
         return joinNode;
       }
     
    -  @Override
    -  public LogicalNode visitTableSubQuery(Set<EvalNode> cnf, LogicalPlan 
plan, LogicalPlan.QueryBlock block,
    -                                        TableSubQueryNode node, 
Stack<LogicalNode> stack) throws PlanningException {
    -    List<EvalNode> matched = Lists.newArrayList();
    -    for (EvalNode eval : cnf) {
    -      if (LogicalPlanner.checkIfBeEvaluatedAtRelation(block, eval, node)) {
    -        matched.add(eval);
    +  private Map<EvalNode, EvalNode> 
transformEvalsWidthByPassNode(Collection<EvalNode> originEvals, LogicalPlan 
plan,
    +                                                                
LogicalPlan.QueryBlock block,
    +                                                                
LogicalNode node, LogicalNode childNode) throws PlanningException {
    +    // transformed -> workingEvals
    +    Map<EvalNode, EvalNode> transformedMap = new HashMap<EvalNode, 
EvalNode>();
    +
    +    if (originEvals.isEmpty()) {
    +      return transformedMap;
    +    }
    +
    +    if (node.getType() == NodeType.UNION) {
    +      // Node가 Union이면 Eval 컬럼은 모두 Simple Name이고 
Child의 OutSchema의 Simple Name과 모두 매칭
    --- End diff --
    
    Could you remove the korean comment?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---