[HIVE-22499] LLAP: Add an EncodedReaderOptions to extend ORC impl for options

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1
• Component/s: llap, ORC
• Labels: pull-request-available
• Assignee: Mustafa İman
• Reporter: Gopal Vijayaraghavan
• Created: 15/Nov/19 06:50
• Updated: 17/Nov/22 08:50
• Resolved: 04/Dec/19 18:14

Description

ORC-570 is an ABI change to the way getFileSystem() by adding an another exception to the implementation.

To accept and use that change requires waiting for an ORC release, while this patch serves the same purpose though falls back for a retry for FileSystem.get() in case the supplier fails at runtime.

Also as a side-note, the FS.get() call is always used in the cases where the file is not being read from a cache such as EncodedOrcFile (so the upstream API change might be overkill).

Issue Links

• requires: ORC-475 ORC reader should lazily get filesystem
• links to: GitHub Pull Request #852