smengcl opened a new pull request, #11614: URL: https://github.com/apache/ozone/pull/11614
Generated-by: Claude Code (Opus 5.5) ## What changes were proposed in this pull request? `BasicOzoneFileSystem#getFileBlockLocations(FileStatus, long, long)` and the same method in `BasicRootedOzoneFileSystem` return every block of the file when given a `LocatedFileStatus`, ignoring `start` and `len`. HDFS, and the default `FileSystem` implementation, return only the blocks that overlap the requested range. Callers that ask about part of a file (for example HBase computing locality for a split reference file, or split planning in MapReduce and Spark) see blocks outside that range. This change returns only the blocks that overlap `[start, start + len)`, or the block containing `start` when `len` is 0, matching HDFS. A negative `start` or `len` throws `IllegalArgumentException`, as in `FileSystem`. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-17111 ## How was this patch tested? New parameterized unit test `TestBasicOzoneFileSystems#fileBlockLocationsHonorRange` covering both filesystems: full range, `Long.MAX_VALUE` length, ranges spanning a block boundary, zero length, start at end of file, and negative arguments. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
