[ 
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)

Reply via email to