Skip to content

python(bug): surface get_data page_size parameter#690

Merged
adamtoth-dev merged 5 commits into
mainfrom
python/pagination-size-alignment
Jul 20, 2026
Merged

python(bug): surface get_data page_size parameter#690
adamtoth-dev merged 5 commits into
mainfrom
python/pagination-size-alignment

Conversation

@adamtoth-dev

@adamtoth-dev adamtoth-dev commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Exposes page_size on ChannelsAPIAsync.get_data so callers can tune how many data points are fetched per gRPC request (default 10,000, server cap 1,000,000).
  • Raises the gRPC max receive message length to 50 MB to accommodate large pages — without this, responses near the server cap would hit the default 4 MB limit and fail.

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Python docs preview: https://sift-stack.github.io/sift/python/pr-690/

Deployed from 20d7718. The link may take up to a minute to become live as GitHub Pages propagates.

Comment thread python/lib/sift_client/resources/channels.py
alexluck-sift
alexluck-sift previously approved these changes Jul 20, 2026
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
adamtoth-dev and others added 3 commits July 20, 2026 15:33
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…_data_as_arrow

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@adamtoth-dev
adamtoth-dev merged commit 629be3a into main Jul 20, 2026
26 checks passed
@adamtoth-dev
adamtoth-dev deleted the python/pagination-size-alignment branch July 20, 2026 23:09
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.

2 participants