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

Tajo QA commented on TAJO-1573:
-------------------------------

{color:red}*-1 overall.*{color}  Here are the results of testing the latest 
attachment 
  
http://issues.apache.org/jira/secure/attachment/12726593/TAJO-1573.Hyun.150421.0.patch.txt
  against master revision release-0.9.0-rc0-646-g6debe7f.

      {color:red}-1 patch.{color}  The patch command could not apply the patch.

    {color:red}-1 patch.{color}  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/901//console

This message is automatically generated.

> Fix addColumn bug (fieldsByName should maintain a list per a simple field 
> name)
> -------------------------------------------------------------------------------
>
>                 Key: TAJO-1573
>                 URL: https://issues.apache.org/jira/browse/TAJO-1573
>             Project: Tajo
>          Issue Type: Bug
>    Affects Versions: 0.10.0
>            Reporter: Dongjoon Hyun
>            Assignee: Dongjoon Hyun
>            Priority: Trivial
>         Attachments: TAJO-1573.Hyun.150420.0.patch.txt, 
> TAJO-1573.Hyun.150421.0.patch.txt
>
>
> In Schema.java, addColumn should maintain a column list per a simple field 
> name. Currently, the maintained list has only the last appended one.
> {code}
> -    fieldsByName.put(newCol.getSimpleName(), TUtil.newList(fields.size() - 
> 1));
> +    if (fieldsByName.containsKey(newCol.getSimpleName())) {
> +      fieldsByName.get(newCol.getSimpleName()).add(fields.size() - 1);
> +    } else {
> +      fieldsByName.put(newCol.getSimpleName(), TUtil.newList(fields.size() - 
> 1));
> +    }
> {code}



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

Reply via email to