You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by ha...@apache.org on 2012/05/25 08:44:31 UTC
svn commit: r1342516 - in
/hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common:
CHANGES.txt src/main/java/org/apache/hadoop/io/Text.java
src/test/java/org/apache/hadoop/io/TestText.java
Author: harsh
Date: Fri May 25 06:44:31 2012
New Revision: 1342516
URL: http://svn.apache.org/viewvc?rev=1342516&view=rev
Log:
svn merge -c 1342514 from trunk. Backports HADOOP-8323 (harsh).
Modified:
hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt (contents, props changed)
hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/Text.java
hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java
Modified: hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt?rev=1342516&r1=1342515&r2=1342516&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt (original)
+++ hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt Fri May 25 06:44:31 2012
@@ -24,6 +24,8 @@ Release 2.0.1-alpha - UNRELEASED
HADOOP-8403. bump up POMs version to 2.0.1-SNAPSHOT. (tucu)
+ HADOOP-8323. Add javadoc and tests for Text.clear() behavior (harsh)
+
BUG FIXES
HADOOP-8372. NetUtils.normalizeHostName() incorrectly handles hostname
Propchange: hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/CHANGES.txt
------------------------------------------------------------------------------
Merged /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt:r1342514
Modified: hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/Text.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/Text.java?rev=1342516&r1=1342515&r2=1342516&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/Text.java (original)
+++ hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/Text.java Fri May 25 06:44:31 2012
@@ -236,6 +236,11 @@ public class Text extends BinaryComparab
/**
* Clear the string to empty.
+ *
+ * <em>Note</em>: For performance reasons, this call does not clear the
+ * underlying byte array that is retrievable via {@link #getBytes()}.
+ * In order to free the byte-array memory, call {@link #set(byte[])}
+ * with an empty byte array (For example, <code>new byte[0]</code>).
*/
public void clear() {
length = 0;
Modified: hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java
URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java?rev=1342516&r1=1342515&r2=1342516&view=diff
==============================================================================
--- hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java (original)
+++ hadoop/common/branches/branch-2/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/TestText.java Fri May 25 06:44:31 2012
@@ -241,6 +241,30 @@ public class TestText extends TestCase {
Text.validateUTF8(utf8, 0, length);
}
+ public void testClear() throws Exception {
+ // Test lengths on an empty text object
+ Text text = new Text();
+ assertEquals(
+ "Actual string on an empty text object must be an empty string",
+ "", text.toString());
+ assertEquals("Underlying byte array length must be zero",
+ 0, text.getBytes().length);
+ assertEquals("String's length must be zero",
+ 0, text.getLength());
+
+ // Test if clear works as intended
+ text = new Text("abcd\u20acbdcd\u20ac");
+ int len = text.getLength();
+ text.clear();
+ assertEquals("String must be empty after clear()",
+ "", text.toString());
+ assertTrue(
+ "Length of the byte array must not decrease after clear()",
+ text.getBytes().length >= len);
+ assertEquals("Length of the string must be reset to 0 after clear()",
+ 0, text.getLength());
+ }
+
public void testTextText() throws CharacterCodingException {
Text a=new Text("abc");
Text b=new Text("a");