Skip to content

Comments

feat(rust): add ForyStreamBuf infrastructure (Phase 1/3)#3341

Closed
Zakir032002 wants to merge 1 commit intoapache:mainfrom
Zakir032002:feat/rust-stream-phase1-buffer-only
Closed

feat(rust): add ForyStreamBuf infrastructure (Phase 1/3)#3341
Zakir032002 wants to merge 1 commit intoapache:mainfrom
Zakir032002:feat/rust-stream-phase1-buffer-only

Conversation

@Zakir032002
Copy link

@Zakir032002 Zakir032002 commented Feb 16, 2026

  • Add standalone ForyStreamBuf with growable buffer
  • Implements fill_buffer for on-demand reading from std::io::Read
  • Buffer grows monotonically without compaction
  • No integration with Reader yet (zero impact on existing code)
  • Includes 4 unit tests for basic functionality

Part 1 of 3-phase implementation.

Why?

What does this PR do?

Related issues

Does this PR introduce any user-facing change?

  • Does this PR introduce any public API change?
  • Does this PR introduce any binary protocol compatibility change?

Benchmark

- Add standalone ForyStreamBuf with growable buffer
- Implements fill_buffer for on-demand reading from std::io::Read
- Buffer grows monotonically without compaction
- No integration with Reader yet (zero impact on existing code)
- Includes 4 unit tests for basic functionality

Design follows C++ PR apache#3307 and addresses apache#3300.
Part 1 of 3-phase implementation.
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.

1 participant