Issue Details (XML | Word | Printable)

Key: HADOOP-4645
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Kan Zhang
Reporter: Owen O'Malley
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

hdfs proxy doesn't package itself right.

Created: 12/Nov/08 10:09 PM   Updated: 23/Apr/09 07:17 PM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 0.20.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works 4645-1.patch 2008-11-13 07:09 PM Kan Zhang 3 kB

Hadoop Flags: Reviewed
Resolution Date: 13/Nov/08 10:58 PM


 Description  « Hide
Currently "ant clean package" creates a directory structure like:

ls build/hadoop-0.20-dev/contrib/hdfsproxy:

classes/
examples/
hdfsproxy-1.0/
hdfsproxy-1.0.jar
test/

and the better structure would copy just the hdfsproxy-1.0 into the hadoop package.



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Kan Zhang added a comment - 13/Nov/08 07:24 AM
patch uploaded.

Hadoop QA added a comment - 13/Nov/08 10:35 PM
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12393883/4645-1.patch
against trunk revision 713783.

+1 @author. The patch does not contain any @author tags.

-1 tests included. The patch doesn't appear to include any new or modified tests.
Please justify why no tests are needed for this patch.

+1 javadoc. The javadoc tool did not generate any warning messages.

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

+1 findbugs. The patch does not introduce any new Findbugs warnings.

+1 Eclipse classpath. The patch retains Eclipse classpath integrity.

+1 core tests. The patch passed core unit tests.

+1 contrib tests. The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3586/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3586/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3586/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/3586/console

This message is automatically generated.


Owen O'Malley added a comment - 13/Nov/08 10:58 PM
I just committed this. Thanks, Kan!

Hudson added a comment - 14/Nov/08 06:33 PM
Integrated in Hadoop-trunk #661 (See http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/661/)
. Package HdfsProxy contrib project without the extra level
of directories. (Kan Zhang via omalley)