[
https://issues.apache.org/jira/browse/HADOOP-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14732068#comment-14732068
]
Jagadesh Kiran N commented on HADOOP-4258:
------------------------------------------
Thanks [~aw] , i will address these three issues ,further any changes please
suggest me
{code}
reapply is too late. This should be in postcheckout. Doing that also simplifies
the code since returning 1 for failure will trigger all the cleanup, etc., so
that wouldn't have to be done in the plugin.
I suspect the problem that Owen O'Malley is actually talking about is a problem
where a new file was added because it conflicted with a pre-existing file in
the source tree. This patch would miss that situation
This seems like it would be very expensive on large patches due to the number
of calls to tr. Off the top of my head, it looks like it could be optimized to
use uniq -id first to limit the set of files that need to looked be at for at
least one of the loops (outer would likely be better, given point 2 above
should be handled prior to even running the inner loop!) {code}
> the test patch script should check for filenames that differ only in case
> -------------------------------------------------------------------------
>
> Key: HADOOP-4258
> URL: https://issues.apache.org/jira/browse/HADOOP-4258
> Project: Hadoop Common
> Issue Type: Sub-task
> Components: build, test
> Reporter: Owen O'Malley
> Assignee: Jagadesh Kiran N
> Labels: test-patch
> Attachments: HADOOP-4258.001.patch, HADOOP-4258.HADOOP-12111.00.patch
>
>
> It would be nice if the test patch script warned about filenames that differ
> only in case. We recently had a patch committed that had a pair of colliding
> filenames and subversion broke badly on my Mac.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
