You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by "Xuanwo (via GitHub)" <gi...@apache.org> on 2026/09/11 17:00:59 UTC

[PR] fix(services/goosefs): return metadata from read streams [opendal]

Xuanwo opened a new pull request, #8283:
URL: https://github.com/apache/opendal/pull/8283

   # Which issue does this PR close?
   
   Related to #7427 (GooseFS support for downstream projects such as Lance).
   
   # Rationale for this change
   
   GooseFS read streams return `RpRead::default()`, so `stream.metadata()` fails with `Unsupported` even though the SDK reader already holds the file's `FileInfo`. This makes `object_store_opendal::get_opts` fall back to a separate stat, including for bounded reads used by Lance.
   
   # What changes are included in this PR?
   
   Open the SDK reader when opening the OpenDAL stream, return its `FileInfo` as `RpRead` metadata, and retain that same reader for the body. Preserve terminal EOF behavior and the existing length lookup for offset-only reads.
   
   The regression test checks full-file metadata and body contents for full, bounded, offset-only, and suffix reads. Against the repository's real GooseFS 2.0.0 Docker fixture with `goosefs-sdk` 0.2.1, it fails on base `43d87279b8af7af96402fd975c40344493c6497b` with `read metadata is not available` and passes with this change. All 119 GooseFS behavior tests also pass.
   
   # Are there any user-facing changes?
   
   GooseFS streams expose full-file size, ETag, and modification time before consuming the body. Full and bounded reads can supply this metadata without a separate stat for the integration fallback.
   
   # Breaking changes
   
   # AI Usage Statement
   
   - Harness: Codex desktop. Model: GPT-6; exact model identifier and reasoning effort are not exposed in this session. Role: Assisted with the implementation and validation against the local GooseFS fixture following maintainer review of the problem and requested fix.
   


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(services/goosefs): return metadata from read streams [opendal]

Posted by "Xuanwo (via GitHub)" <gi...@apache.org>.
Xuanwo merged PR #8283:
URL: https://github.com/apache/opendal/pull/8283


-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org