You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/01/29 01:46:39 UTC
[jira] [Commented] (TAJO-1573) Fix addColumn bug (fieldsByName
should maintain a list per a simple field name)
[ https://issues.apache.org/jira/browse/TAJO-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15122634#comment-15122634 ]
ASF GitHub Bot commented on TAJO-1573:
--------------------------------------
Github user dongjoon-hyun closed the pull request at:
https://github.com/apache/tajo/pull/545
> 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)