Skip to content

Update ARCHITECTURE.md to reflect current implementation#10

Merged
jhamman merged 1 commit into
mainfrom
update-architecture-docs
Mar 15, 2026
Merged

Update ARCHITECTURE.md to reflect current implementation#10
jhamman merged 1 commit into
mainfrom
update-architecture-docs

Conversation

@jhamman

@jhamman jhamman commented Mar 15, 2026

Copy link
Copy Markdown
Member

Summary

  • HTTP storage: Documented SimpleHttpStore (CDN-compatible, tolerates missing Content-Length) replacing references to zarrs_http::HTTPStore
  • Consolidated metadata: Moved from Future Work to its own implemented section; documented both V2 .zmetadata and V3 inline consolidated_metadata support, plus the zarrsJlStorageGet FFI function
  • Cargo features: Updated to match actual Cargo.toml (only icechunk is feature-gated; filesystem/HTTP/S3/GCS are unconditional)
  • Storage dispatch: Updated code snippet to show current URL pipeline implementation
  • S3/GCS: Moved from Future Work to implemented; only Azure remains as future
  • FFI description: Updated to reflect that zarrs_jl depends on zarrs directly, not zarrs_ffi
  • Package structure: Fixed Rust source files (lib.rs, http_store.rs, icechunk.rs), added url_pipeline.jl and new test files

Closes #2

Test plan

  • Verify all code snippets match actual source files
  • Confirm Future Work section only lists genuinely unimplemented features
  • Review package structure listing against actual file tree

🤖 Generated with Claude Code

- Document SimpleHttpStore replacing zarrs_http (CDN Content-Length compat)
- Move consolidated metadata from Future Work to implemented section
- Document zarrsJlStorageGet FFI function for consolidated metadata
- Update Cargo features to match actual Cargo.toml (only icechunk gated)
- Move S3/GCS from Future Work to implemented, keep Azure as future
- Update FFI description: zarrs_jl depends on zarrs directly, not zarrs_ffi
- Fix package structure: actual Rust files (lib.rs, http_store.rs, icechunk.rs)
- Add url_pipeline.jl and new test files to listings
- Update storage dispatch code to show current URL pipeline implementation

Closes #2

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jhamman
jhamman merged commit 2a53e82 into main Mar 15, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update ARCHITECTURE.md to reflect current implementation

1 participant