[ https://issues.apache.org/jira/browse/HIVE-20847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16679184#comment-16679184 ]
Hive QA commented on HIVE-20847: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12947237/HIVE-20847.1.patch {color:red}ERROR:{color} -1 due to no test(s) being added or modified. {color:red}ERROR:{color} -1 due to 10 failed/errored test(s), 15524 tests executed *Failed tests:* {noformat} TestMiniDruidCliDriver - did not produce a TEST-*.xml file (likely timed out) (batchId=196) [druidmini_masking.q,druidmini_test1.q,druidkafkamini_basic.q,druidmini_joins.q,druid_timestamptz.q] org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testCancelRenewTokenFlow (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testConnection (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testIsValid (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testIsValidNeg (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testNegativeProxyAuth (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testNegativeTokenAuth (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testProxyAuth (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testRenewDelegationToken (batchId=273) org.apache.hive.minikdc.TestJdbcWithDBTokenStore.testTokenAuth (batchId=273) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/14800/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/14800/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-14800/ 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: 10 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12947237 - PreCommit-HIVE-Build > Review of NullScan Code > ----------------------- > > Key: HIVE-20847 > URL: https://issues.apache.org/jira/browse/HIVE-20847 > Project: Hive > Issue Type: Improvement > Components: Physical Optimizer > Affects Versions: 3.1.0, 4.0.0 > Reporter: BELUGA BEHR > Assignee: BELUGA BEHR > Priority: Minor > Attachments: HIVE-20847.1.patch, HIVE-20847.1.patch > > > What got me looking at this class was the verboseness of some of the logging. > I would like to request that we DEBUG the logging since this level of detail > means nothing to a cluster admin. > Also... this {{contains}} call would be better applied onto a {{HashSet}} > instead of an {{ArrayList}}. > {code:java|title=NullScanTaskDispatcher.java} > private void processAlias(MapWork work, Path path, ArrayList<String> > aliasesAffected, ArrayList<String> aliases) { > // the aliases that are allowed to map to a null scan. > ArrayList<String> allowed = new ArrayList<String>(); > for (String alias : aliasesAffected) { > if (aliases.contains(alias)) { > allowed.add(alias); > } > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)