[HIVE-23533] Remove an FS#exists call from AcidUtils#getLogicalLength

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1
• Assignee: Karen Coppage Assign to me
• Reporter: Karen Coppage
• Created: 22/May/20 14:07
• Updated: 17/Nov/22 08:48
• Resolved: 04/Jun/20 08:34

Description

 Path lengths = OrcAcidUtils.getSideFile(file.getPath());
    if(!fs.exists(lengths)) {
...
      return file.getLen();
    }
    long len = OrcAcidUtils.getLastFlushLength(fs, file.getPath());

OrcAcidUtils.getLastFlushLength also has an exists() check and returns Long.MAX_VALUE if false.
exists() is expensive on S3.

Issue Links

• is cloned by: HIVE-26232 AcidUtils getLogicalLength shouldn't be called for external tables