[HADOOP-7131] set() and toString Methods of the org.apache.hadoop.io.Text class does not include the root exception, in the wrapping RuntimeException.

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.20.1, 0.20.2
• Fix Version/s: 0.23.0
• Component/s: io
• Hadoop Flags: Reviewed
• Assignee: Uma Maheswara Rao G
• Reporter: Uma Maheswara Rao G
• Created: 03/Feb/11 16:18
• Updated: 15/Nov/11 00:50
• Resolved: 07/Mar/11 03:16

Description

In below code snippets, we can include e, instead of e.toString(), so that caller can get complete trace.

1)
/** Set to contain the contents of a string.
*/
public void set(String string) {
try

{ ByteBuffer bb = encode(string, true); bytes = bb.array(); length = bb.limit(); }

catch(CharacterCodingException e)

{ throw new RuntimeException("Should not have happened ",e.toString()); }
}
2)
public String toString() {
try { return decode(bytes, 0, length); } catch (CharacterCodingException e) { throw new RuntimeException("Should not have happened ",e.toString()); }

}