[
https://issues.apache.org/jira/browse/TAJO-1573?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14514148#comment-14514148
]
ASF GitHub Bot commented on TAJO-1573:
--------------------------------------
Github user jihoonson commented on a diff in the pull request:
https://github.com/apache/tajo/pull/545#discussion_r29146884
--- Diff: tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
---
@@ -1200,8 +1200,8 @@ private static EvalNode
getNaturalJoinCondition(JoinNode joinNode) {
Column rightJoinKey;
for (Column common : commons.getColumns()) {
- leftJoinKey = leftSchema.getColumn(common.getQualifiedName());
- rightJoinKey = rightSchema.getColumn(common.getQualifiedName());
+ leftJoinKey =
leftSchema.getColumn(leftSchema.getColumnIdByName(common.getQualifiedName()));
--- End diff --
Would you explain why you changed this line?
> 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)
