[ https://issues.apache.org/jira/browse/HIVE-22819?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17029842#comment-17029842 ]
Hive QA commented on HIVE-22819: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12992578/HIVE-22819.2.patch {color:green}SUCCESS:{color} +1 due to 19 test(s) being added or modified. {color:green}SUCCESS:{color} +1 due to 17965 tests passed Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/20456/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/20456/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-20456/ 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 {noformat} This message is automatically generated. ATTACHMENT ID: 12992578 - 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 > > > {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)