[HADOOP-16951] Tidy Up Text and ByteWritables Classes

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: common
• Target Version/s: 3.4.0
• Hadoop Flags: Reviewed
• Assignee: David Mollitor
• Reporter: David Mollitor
• Created: 01/Apr/20 23:22
• Updated: 27/Jan/24 06:30
• Resolved: 17/Apr/20 15:16

Description

1. Remove superfluous code
2. Remove superfluous comments
3. Checkstyle fixes
4. Remove methods that simply call super.method()
5. Use Java 8 facilities to streamline code where applicable
6. Simplify and unify some of the constructs between the two classes

 

The one meaningful change is that I am suggesting that the expanding of the arrays be 1.5x instead of 2x per expansion.  I pulled this idea from open JDK.

Issue Links

• causes: HADOOP-17901 Performance degradation in Text.append() after HADOOP-16951
• links to: GitHub Pull Request #1932