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

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

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

    https://github.com/apache/tajo/pull/17#discussion_r13454181
  
    --- Diff: 
tajo-core/src/main/java/org/apache/tajo/master/DefaultTaskScheduler.java ---
    @@ -821,7 +820,7 @@ public void 
assignToLeafTasks(LinkedList<TaskRequestEvent> taskRequests) {
                   host, container.getTaskPort()));
               assignedRequest.add(attemptId);
     
    -          scheduledObjectNum -= task.getAllFragments().size();
    --- End diff --
    
    If it is changed to minus one, LazyTaskScheduler will not work anymore. 
@jihoonson Could you check this? 
    
    I think LazyTaskScheduler already does not work now in all query cases. We 
need to decide if we will maintain LazyTaskScheduler continuously or remove it 
and refactor the code around schedulers.


> If all tables participate in the BROADCAST JOIN, there is some missing data.
> ----------------------------------------------------------------------------
>
>                 Key: TAJO-839
>                 URL: https://issues.apache.org/jira/browse/TAJO-839
>             Project: Tajo
>          Issue Type: Bug
>            Reporter: Hyoungjun Kim
>            Assignee: Hyoungjun Kim
>            Priority: Minor
>
> See title. 
> If all tables participate in the BROADCAST JOIN and table has several files, 
> there is some missing data.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to