[
https://issues.apache.org/jira/browse/YETUS-281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15135880#comment-15135880
]
Allen Wittenauer commented on YETUS-281:
----------------------------------------
Found the bug. hadoop_check_module_present never declares module as a local
variable. So it overwrites the one in the main loop, thus explaining the
duplication and the dropping. Even with that simple fix, it's still better to
use the Yetus mainline code here since:
a) it uses maven to determine the dependency ordering
b) significantly easier to maintain
> hadoop: use built-in dependency order
> -------------------------------------
>
> Key: YETUS-281
> URL: https://issues.apache.org/jira/browse/YETUS-281
> Project: Yetus
> Issue Type: Improvement
> Components: Test Patch
> Reporter: Allen Wittenauer
> Assignee: Allen Wittenauer
> Priority: Blocker
> Attachments: YETUS-281.00.patch
>
>
> With the commit of YETUS-280, hadoop should loosen the reins a bit and drop
> several hundred lines of code to use the built-in dependency creator.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
