Skip to content

APP-16465: Add SequencesByDatasetID RPC#855

Merged
vpandiarajan20 merged 2 commits into
mainfrom
APP-16465-sequences-by-dataset-id-proto
May 20, 2026
Merged

APP-16465: Add SequencesByDatasetID RPC#855
vpandiarajan20 merged 2 commits into
mainfrom
APP-16465-sequences-by-dataset-id-proto

Conversation

@vpandiarajan20
Copy link
Copy Markdown
Member

@vpandiarajan20 vpandiarajan20 commented May 19, 2026

Summary

  • Adds SequencesByDatasetID to DataService. Pagination mirrors ListSequences: dataset_id + page_size + page_token in, sequences + next_page_token out.

Test plan

  • buf lint clean
  • Regen via ready-for-protos label
  • Verified end-to-end via App PR

🤖 Generated with Claude Code

@vpandiarajan20 vpandiarajan20 added the ready-for-protos add this when you want protos to compile on every commit label May 19, 2026
@vpandiarajan20 vpandiarajan20 force-pushed the APP-16465-sequences-by-dataset-id-proto branch from 5296663 to 123e9bb Compare May 19, 2026 19:59
@vpandiarajan20 vpandiarajan20 changed the base branch from APP-16463-sequence-dataset-ids-proto to main May 19, 2026 19:59
@vpandiarajan20 vpandiarajan20 added ready-for-protos add this when you want protos to compile on every commit safe to test committer is a member of this org and removed ready-for-protos add this when you want protos to compile on every commit labels May 19, 2026
@vpandiarajan20 vpandiarajan20 requested a review from njooma May 19, 2026 20:57
Pagination shape mirrors ListSequences: dataset_id + page_size + page_token in,
sequences + next_page_token out.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vpandiarajan20 vpandiarajan20 force-pushed the APP-16465-sequences-by-dataset-id-proto branch from 0fa41c5 to c47d087 Compare May 20, 2026 17:51
@vpandiarajan20 vpandiarajan20 removed the ready-for-protos add this when you want protos to compile on every commit label May 20, 2026
@vpandiarajan20 vpandiarajan20 added the ready-for-protos add this when you want protos to compile on every commit label May 20, 2026
@vpandiarajan20 vpandiarajan20 removed the ready-for-protos add this when you want protos to compile on every commit label May 20, 2026
@vpandiarajan20 vpandiarajan20 merged commit b5d9927 into main May 20, 2026
3 checks passed
@vpandiarajan20 vpandiarajan20 deleted the APP-16465-sequences-by-dataset-id-proto branch May 20, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

protos-compiled safe to test committer is a member of this org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants