You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by tu...@apache.org on 2012/09/20 19:41:41 UTC
svn commit: r1388130 - in /oozie/trunk: bin/test-patch-05-patch-raw-analysis
release-log.txt
Author: tucu
Date: Thu Sep 20 17:41:41 2012
New Revision: 1388130
URL: http://svn.apache.org/viewvc?rev=1388130&view=rev
Log:
OOZIE-998 test-patch doesn't allow lines that are exactly 132 characters (rkanter via tucu)
Modified:
oozie/trunk/bin/test-patch-05-patch-raw-analysis
oozie/trunk/release-log.txt
Modified: oozie/trunk/bin/test-patch-05-patch-raw-analysis
URL: http://svn.apache.org/viewvc/oozie/trunk/bin/test-patch-05-patch-raw-analysis?rev=1388130&r1=1388129&r2=1388130&view=diff
==============================================================================
--- oozie/trunk/bin/test-patch-05-patch-raw-analysis (original)
+++ oozie/trunk/bin/test-patch-05-patch-raw-analysis Thu Sep 20 17:41:41 2012
@@ -101,7 +101,8 @@ checkNoTrailingSpaces() {
}
###############################################################################
checkLinesLength() {
- longLines=`grep "^+ " ${PATCHFILE} | awk 'BEGIN{count=0}{if ( length > 132 ) { count=count+1} }END{ print count}'`
+ # We check for > 133 to account for the "+" sign
+ longLines=`grep "^+ " ${PATCHFILE} | awk 'BEGIN{count=0}{if ( length > 133 ) { count=count+1} }END{ print count}'`
if [[ ${longLines} != 0 ]] ; then
REPORT+=("-1 the patch contains ${longLines} line(s) longer than 132 characters")
else
Modified: oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1388130&r1=1388129&r2=1388130&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Thu Sep 20 17:41:41 2012
@@ -1,5 +1,6 @@
-- Oozie 3.4.0 release (trunk - unreleased)
+OOZIE-998 test-patch doesn't allow lines that are exactly 132 characters (rkanter via tucu)
OOZIE-739 a coord action fails because the uri points to a namenode that is not in whitelist. the E0901 error shows in the oozie.log, but not written to the database (mona,mbattisha via virag)
OOZIE-989 Testcases failing intermittently where coordinator jobs are in catchup mode (virag)
OOZIE-990 TestLogStreamer.testStreamLog fails in very rare cases (rkanter via tucu)