|
[
Permlink
| « Hide
]
Tsz Wo (Nicholas), SZE added a comment - 23/Apr/09 12:35 AM
5711_20090422.patch: change log level
Tsz Wo (Nicholas), SZE made changes - 23/Apr/09 12:35 AM
Maybe a more informative message is better, something like "file xx is closed by yy".
5711_20090428.patch: changed the message.
Tsz Wo (Nicholas), SZE made changes - 29/Apr/09 12:29 AM
Tsz Wo (Nicholas), SZE made changes - 29/Apr/09 05:49 PM
[exec] -1 overall.
[exec]
[exec] +1 @author. The patch does not contain any @author tags.
[exec]
[exec] -1 tests included. The patch doesn't appear to include any new or modified tests.
[exec] Please justify why no tests are needed for this patch.
[exec]
[exec] +1 javadoc. The javadoc tool did not generate any warning messages.
[exec]
[exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings.
[exec]
[exec] +1 findbugs. The patch does not introduce any new Findbugs warnings.
[exec]
[exec] +1 Eclipse classpath. The patch retains Eclipse classpath integrity.
[exec]
[exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.
The change is trivial. Don't want to waste cycles in Hudson. Will commit it soon.
I have committed this to 0.20.1 and above.
Tsz Wo (Nicholas), SZE made changes - 29/Apr/09 08:53 PM
Integrated in Hadoop-trunk #822 (See http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/822/
. Change Namenode file close log to info.
Owen O'Malley made changes - 08/Jul/09 04:43 PM
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||