[DRILL-5603] Replace String file paths to Hadoop Path

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.10.0, 1.15.0
• Fix Version/s: 1.16.0
• Labels: ready-to-commit
• Assignee: Vitalii Diravka
• Reporter: Vitalii Diravka
• Reviewer: Vova Vysotskyi
• Created: 22/Jun/17 15:31
• Updated: 05/Mar/19 15:41
• Resolved: 05/Mar/19 15:41

Description

String paths are used in the different places of the Drill code:
FileSelection, Metadata, ParquetGroupScan, ReadEntryWithPath, FileWork, FormatSelection, FormatPlugin, PartitionLocation and so on.
Code with paths as strings is hard to test and maintain. And there are a lot of String to Path conversions and vice versa.

The aim of this jira to replace String paths with Hadoop Path. The job of the Path class is do properly implement file path operations, just as the job of the older File and newer Path classes in Java is to handle OS paths.

For the serialized structures a custom JsonSerializer<Path> and JsonDeserializer<Path> can be implemented.

Issue Links

• is required by: DRILL-6552 Drill Metadata management "Drill Metastore"
• relates to: DRILL-3867 Store relative paths in metadata file
• links to: GitHub Pull Request #1657