Issue Details (XML | Word | Printable)

Key: HADOOP-3266
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Hemanth Yamijala
Reporter: Hemanth Yamijala
Votes: 0
Watchers: 0
Operations

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

Remove HOD changes from CHANGES.txt, as they are now inside src/contrib/hod

Created: 17/Apr/08 08:13 AM   Updated: 20/Oct/08 09:29 PM
Return to search
Component/s: contrib/hod
Affects Version/s: 0.17.0, 0.18.0
Fix Version/s: 0.17.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works CHANGES-0.17.patch 2008-04-17 08:18 AM Hemanth Yamijala 7 kB
Text File Licensed for inclusion in ASF works CHANGES-trunk.patch 2008-04-17 08:18 AM Hemanth Yamijala 7 kB

Hadoop Flags: Reviewed, Incompatible change
Release Note: Moved HOD change items from CHANGES.txt to a new file src/contrib/hod/CHANGES.txt.
Resolution Date: 17/Apr/08 10:01 AM


 Description  « Hide
HOD changes were originally documented in the CHANGES.txt file.

However, to assist ease of commiting patches - as hod committers (me) don't have access to the main CHANGES.txt file, and also because HOD is typically deployed and installed to separate locations, it made sense to have a separate CHANGES.txt for HOD. This is already present under src/contrib/hod. This issue is to remove the older entries in the main CHANGES.txt, so there is no duplication.



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Hemanth Yamijala added a comment - 17/Apr/08 08:18 AM
The attached files remove entries related to HOD from the main CHANGES.txt file.

This patch may fail Hudson as no test cases are required, and also if other modifications are made to CHANGES.txt.


Hadoop QA added a comment - 17/Apr/08 09:44 AM
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12380373/CHANGES-trunk.patch
against trunk revision 645773.

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

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

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

javac +1. The applied patch does not generate any new javac compiler warnings.

release audit +1. The applied patch does not generate any new release audit warnings.

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

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

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

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

This message is automatically generated.


Devaraj Das added a comment - 17/Apr/08 10:01 AM
I just committed this. Thanks, Hemanth!

Hudson added a comment - 17/Apr/08 12:11 PM