[jira] [Commented] (TAJO-1987) Add stream API to Projectable and refactor its usage codes

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TAJO-1987?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15037350#comment-15037350 ] 

ASF GitHub Bot commented on TAJO-1987:
--------------------------------------

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

    https://github.com/apache/tajo/pull/874#discussion_r46516484
  
    --- Diff: tajo-plan/src/main/java/org/apache/tajo/plan/logical/ProjectionNode.java ---
    @@ -64,8 +65,13 @@ public void setTargets(List<Target> targets) {
       public List<Target> getTargets() {
         return this.targets;
       }
    -	
    -	public void setChild(LogicalNode subNode) {
    +
    +  @Override
    +  public Stream<Target> targets() {
    --- End diff --
    
    This is because we still need List allowing access elements by index in some codes.


> Add stream API to Projectable and refactor its usage codes
> ----------------------------------------------------------
>
>                 Key: TAJO-1987
>                 URL: https://issues.apache.org/jira/browse/TAJO-1987
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: Planner/Optimizer
>            Reporter: Hyunsik Choi
>            Assignee: Hyunsik Choi
>            Priority: Minor
>             Fix For: 0.12.0
>
>
> This patch is a trivial patch to add a stream API to Projectable and refactor its usage codes.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)