[ https://issues.apache.org/jira/browse/HIVE-23088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17087192#comment-17087192 ]
Hive QA commented on HIVE-23088: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/13000446/HIVE-23088.01.patch {color:green}SUCCESS:{color} +1 due to 2 test(s) being added or modified. {color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 17137 tests executed *Failed tests:* {noformat} org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[schema_evol_par_vec_table_dictionary_encoding] (batchId=78) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/21780/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/21780/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-21780/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.YetusPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 1 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 13000446 - PreCommit-HIVE-Build > Using Strings from log4j breaks non-log4j users > ----------------------------------------------- > > Key: HIVE-23088 > URL: https://issues.apache.org/jira/browse/HIVE-23088 > Project: Hive > Issue Type: Bug > Affects Versions: 3.1.2 > Reporter: Vova Vysotskyi > Assignee: David Lavati > Priority: Major > Labels: pull-request-available > Fix For: 4.0.0, 3.2.0, 3.1.3 > > Attachments: HIVE-23088.01.branch-3.patch, HIVE-23088.01.patch > > Time Spent: 10m > Remaining Estimate: 0h > > {{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}}. -- This message was sent by Atlassian Jira (v8.3.4#803005)