[
https://issues.apache.org/jira/browse/HIVE-21225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16886324#comment-16886324
]
Jason Dere commented on HIVE-21225:
-----------------------------------
[~gopalv] and [~vgumashta] brought this one to my attention as a fix for
HIVE-22001, since using just a single recursive file listing will prevent the
problem in HIVE-22001 where the directory was listed once, one or more files
are deleted from that directory, and then getAcidState() tries to do a
subsequent listing on a deleted file.
However this patch will need additional modifications as the error in
HIVE-22001 occurred in the file-ids path, which still issues multiple FS calls
in the current patch.
> ACID: getAcidState() should cache a recursive dir listing locally
> -----------------------------------------------------------------
>
> Key: HIVE-21225
> URL: https://issues.apache.org/jira/browse/HIVE-21225
> Project: Hive
> Issue Type: Improvement
> Components: Transactions
> Reporter: Gopal V
> Assignee: Vaibhav Gumashta
> Priority: Major
> Attachments: HIVE-21225.1.patch, HIVE-21225.10.patch,
> HIVE-21225.11.patch, HIVE-21225.12.patch, HIVE-21225.13.patch,
> HIVE-21225.2.patch, HIVE-21225.3.patch, HIVE-21225.4.patch,
> HIVE-21225.4.patch, HIVE-21225.5.patch, HIVE-21225.6.patch,
> HIVE-21225.7.patch, HIVE-21225.7.patch, HIVE-21225.8.patch,
> HIVE-21225.9.patch, async-pid-44-2.svg
>
>
> Currently getAcidState() makes 3 calls into the FS api which could be
> answered by making a single recursive listDir call and reusing the same data
> to check for isRawFormat() and isValidBase().
> All delta operations for a single partition can go against a single listed
> directory snapshot instead of interacting with the NameNode or ObjectStore
> within the inner loop.
--
This message was sent by Atlassian JIRA
(v7.6.14#76016)
