[GEODE-1994] Change geode StringUtils to extend commons StringUtils

• Type: Wish
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: general, management
• Assignee: Patrick Rhomberg
• Reporter: Kirk Lund
• Created: 12/Oct/16 17:40
• Updated: 20/Apr/18 19:52
• Resolved: 18/Sep/17 17:36

Description

org.apache.geode.internal.lang.StringUtils duplicates some of the methods in org.apache.commons.lang.StringUtils with some inconsistencies.

isBlank is implemented identically

isEmpty is inconsistent – commons version returns true if string is null, while geode version returns false if string is null

Issue Links

• is related to: GEODE-2704 Remove Pulse's custom StringUtils in favor of Apache Commons StringUtils
• links to: GitHub Pull Request #521
• links to: GitHub Pull Request #528