[ https://issues.apache.org/jira/browse/HIVE-22819?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17045457#comment-17045457 ]
Hive QA commented on HIVE-22819: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12994628/HIVE-22819.6.patch {color:red}ERROR:{color} -1 due to no test(s) being added or modified. {color:red}ERROR:{color} -1 due to 2 failed/errored test(s), 18073 tests executed *Failed tests:* {noformat} org.apache.hadoop.hive.metastore.TestMetastoreHousekeepingLeaderEmptyConfig.testHouseKeepingThreadExistence (batchId=251) org.apache.hive.jdbc.TestJdbcWithMiniHS2.testParallelCompilation2 (batchId=292) {noformat} Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/20834/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/20834/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-20834/ 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: 2 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12994628 - PreCommit-HIVE-Build > Refactor Hive::listFilesCreatedByQuery to make it faster for object stores > -------------------------------------------------------------------------- > > Key: HIVE-22819 > URL: https://issues.apache.org/jira/browse/HIVE-22819 > Project: Hive > Issue Type: Improvement > Reporter: Marton Bod > Assignee: Marton Bod > Priority: Major > Attachments: HIVE-22819.1.patch, HIVE-22819.2.patch, > HIVE-22819.3.patch, HIVE-22819.4.patch, HIVE-22819.5.patch, HIVE-22819.6.patch > > > {color:#0000ff}Hive::listFilesCreatedByQuery{color} does an exists(), an > isDir() and then a listing call. This can be expensive in object stores. We > should instead directly list the files in the directory (we'd have to handle > an exception if the directory does not exists, but issuing a single call to > the object store would most likely still end up being more performant). -- This message was sent by Atlassian Jira (v8.3.4#803005)