[ 
https://issues.apache.org/jira/browse/HIVE-22962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17050895#comment-17050895
 ] 

Hive QA commented on HIVE-22962:
--------------------------------



Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12995509/HIVE-22962.02.patch

{color:red}ERROR:{color} -1 due to no test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 59 failed/errored test(s), 18096 tests 
executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[ambiguitycheck] 
(batchId=85)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[cast1] (batchId=87)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[decimal_2] (batchId=72)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_aes_decrypt] 
(batchId=62)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_aes_encrypt] 
(batchId=100)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_crc32] (batchId=2)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_decode] (batchId=99)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_md5] (batchId=10)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_sha1] (batchId=8)
org.apache.hadoop.hive.cli.TestCliDriver.testCliDriver[udf_sha2] (batchId=13)
org.apache.hadoop.hive.cli.TestKuduCliDriver.testCliDriver[kudu_queries] 
(batchId=297)
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[vector_decimal_2]
 (batchId=175)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testAlterPartition 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testAlterTable 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testAlterTableCascade 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testAlterViewParititon 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testColumnStatistics 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testComplexTable 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testComplexTypeApi 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testConcurrentMetastores
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testCreateAndGetTableWithDriver
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testCreateTableSettingId
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDBLocationChange 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDBOwner 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDBOwnerChange 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDatabase 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDatabaseLocation 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDatabaseLocationWithPermissionProblems
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDropDatabaseCascadeMVMultiDB
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testDropTable 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testFilterLastPartition 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testFilterSinglePartition
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testFunctionWithResources
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetConfigValue 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetMetastoreUuid 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetPartitionsWithSpec
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetSchemaWithNoClassDefFoundError
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetTableObjects 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testGetUUIDInParallel 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testJDOPersistanceManagerCleanup
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testListPartitionNames 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testListPartitions 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testListPartitionsWihtLimitEnabled
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testNameMethods 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testPartition 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testPartitionFilter 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testPartitionFilterLike 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testRenamePartition 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testRetriableClientWithConnLifetime
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testSimpleFunction 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testSimpleTable 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testSimpleTypeApi 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testStatsFastTrivial 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testSynchronized 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testTableDatabase 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testTableFilter 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testUpdatePartitionStat_doesNotUpdateStats
 (batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testValidateTableCols 
(batchId=307)
org.apache.hive.minikdc.TestRemoteHiveMetaStoreKerberos.testVersion 
(batchId=307)
{noformat}

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/20940/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/20940/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-20940/

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: 59 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12995509 - PreCommit-HIVE-Build

> Reuse HiveRelFieldTrimmer instance across queries
> -------------------------------------------------
>
>                 Key: HIVE-22962
>                 URL: https://issues.apache.org/jira/browse/HIVE-22962
>             Project: Hive
>          Issue Type: Improvement
>          Components: CBO
>            Reporter: Jesus Camacho Rodriguez
>            Assignee: Jesus Camacho Rodriguez
>            Priority: Major
>         Attachments: HIVE-22962.01.patch, HIVE-22962.02.patch, 
> HIVE-22962.patch
>
>
> Currently we create multiple {{HiveRelFieldTrimmer}} instances per query. 
> {{HiveRelFieldTrimmer}} uses a method dispatcher that has a built-in caching 
> mechanism: given a certain object, it stores the method that was called for 
> the object class. However, by instantiating the trimmer multiple times per 
> query and across queries, we create a new dispatcher with each instantiation, 
> thus effectively removing the caching mechanism that is built within the 
> dispatcher.
> This issue is to reutilize the same {{HiveRelFieldTrimmer}} instance within a 
> single query and across queries.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to