[HIVE-23088] Using Strings from log4j breaks non-log4j users

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.2
• Fix Version/s: 3.1.3, 3.2.0, 4.0.0-alpha-1
• Labels: pull-request-available
• Assignee: David Lavati
• Reporter: Vova Vysotskyi
• Created: 27/Mar/20 10:13
• Updated: 21/Oct/22 06:56
• Resolved: 23/Apr/20 16:16

Description

HookUtils uses explicitly org.apache.logging.log4j.util.Strings class from log4j, but it may break clients who use other loggers and should exclude log4j from the classpath.

commons-lang has class StringUtils which may be used as a replacement for this one:
Strings.isBlank -> StringUtils.isBlank.

Issue Links

• links to: GitHub Pull Request #986