[HADOOP-18959] Use builder for prefetch CachingBlockManager

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: fs/s3
• Labels: pull-request-available
• Target Version/s: 3.4.0, 3.3.9
• Hadoop Flags: Reviewed
• Assignee: Viraj Jasani
• Reporter: Viraj Jasani
• Created: 30/Oct/23 03:55
• Updated: 21/Jan/24 03:58
• Resolved: 21/Jan/24 03:54

Description

Some of the recent changes (HADOOP-18399, HADOOP-18291, HADOOP-18829 etc) have added more params for prefetch CachingBlockManager c'tor to process read/write block requests. They have added too many params and more are likely to be introduced later. We should use builder pattern to pass params.

This would also help consolidating required prefetch params into one single place within S3ACachingInputStream, from scattered locations.

Issue Links

• links to: GitHub Pull Request #6240
• links to: GitHub Pull Request #6302