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

Hyunsik Choi commented on TAJO-1422:
------------------------------------

Thanks for your investigation. I'll close this issue as fixed, and then I'll 
make another jira in order to make this code better.

> Investigate the case where fragments == null in SeqScanExec
> -----------------------------------------------------------
>
>                 Key: TAJO-1422
>                 URL: https://issues.apache.org/jira/browse/TAJO-1422
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: Hyunsik Choi
>            Priority: Minor
>             Fix For: 0.11.0
>
>
> See the following part in SeqScanExec. Intuitively, we don't need to make and 
> execute Task if fragments is null. But, the current code requires null check. 
> We need to investigate its essential cause, and we should remove this case if 
> possible.
> {code:title=SeqScanExec.java}
>     if (fragments != null) {
>       if (fragments.length > 1) {
>         this.scanner = new MergeScanner(context.getConf(), 
> plan.getPhysicalSchema(), meta,
>             FragmentConvertor.convert(context.getConf(), fragments), projected
>         );
> {code}



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

Reply via email to