Skip to content

Conversation

@AntoinePrv
Copy link
Member

@AntoinePrv AntoinePrv commented Dec 29, 2025

No description provided.

AntoinePrv and others added 4 commits December 29, 2025 13:59
Change the IPC route parameters from chunk-based to range-based:
- Replace row_chunk_size/row_chunk with start_row/end_row
- Replace col_chunk_size/col_chunk with start_col/end_col

This simplifies the protocol by directly specifying ranges instead of
requiring clients to calculate offsets from chunk indices and sizes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Update the client to use the new range-based protocol:
- Change TableOptions interface to use start_row/end_row/start_col/end_col
- Update fetchTable to send new parameter names to server
- Modify ArrowModel to calculate start/end from chunk indices
- Update test mocks to use new parameters

This completes the protocol migration from chunk-based to range-based
parameters, making the API more intuitive and flexible.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@AntoinePrv AntoinePrv merged commit bd64602 into QuantStack:main Dec 29, 2025
8 checks passed
@AntoinePrv AntoinePrv deleted the url-params branch December 29, 2025 13:55
@AntoinePrv AntoinePrv added the feature Add a new functionality to the project label Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Add a new functionality to the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant