Skip to content

feat(discovery-core): paginated subchannel discovery with cursor support - #461

Merged
m-kus merged 1 commit into
mainfrom
pr-split/07-paginated-subchannels
Feb 17, 2026
Merged

m-kus merged 1 commit into
mainfrom
pr-split/07-paginated-subchannels

Conversation

@m-kus

@m-kus m-kus commented Feb 9, 2026 •

Copy link
Copy Markdown

TL;DR

Added a cursor-based pagination mechanism for subchannel discovery with improved logging and caching.

What changed?

  • Added discover_subchannels_paginated function that builds on the existing discover_subchannels function
  • Implemented cursor-based pagination that tracks discovery progress and caches results
  • Added tracing with debug and trace level logs for subchannel discovery
  • Enhanced the documentation for SubchannelDiscoveryResult.last_index to clarify its usage
  • Added comprehensive tests for the new pagination functionality

This change is Reviewable

m-kus commented Feb 9, 2026 •

Copy link
Copy Markdown
Author

@m-kus
m-kus requested a review from Yoni-Starkware February 9, 2026 20:25
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 9a8ab06 to 97e2373 Compare February 11, 2026 13:05
@m-kus
m-kus force-pushed the pr-split/06-paginated-incoming branch from 3a23638 to f0b6b73 Compare February 11, 2026 15:07
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 97e2373 to 4858b01 Compare February 11, 2026 15:07
@m-kus
m-kus force-pushed the pr-split/06-paginated-incoming branch from f0b6b73 to 88cd85e Compare February 11, 2026 16:08
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 4858b01 to 04787da Compare February 11, 2026 16:08
@m-kus
m-kus force-pushed the pr-split/06-paginated-incoming branch from 88cd85e to cd78718 Compare February 11, 2026 19:10
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 04787da to 6be1674 Compare February 11, 2026 19:10

@Yoni-Starkware Yoni-Starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@Yoni-Starkware reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @m-kus).

@m-kus
m-kus changed the base branch from pr-split/06-paginated-incoming to graphite-base/461 February 13, 2026 01:38
@m-kus
m-kus force-pushed the graphite-base/461 branch from cd78718 to acf24fc Compare February 13, 2026 01:47
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 6be1674 to 24ce52a Compare February 13, 2026 01:47
@m-kus
m-kus changed the base branch from graphite-base/461 to pr-split/06-paginated-incoming February 13, 2026 01:47
@m-kus
m-kus changed the base branch from pr-split/06-paginated-incoming to graphite-base/461 February 16, 2026 12:43
@m-kus
m-kus force-pushed the graphite-base/461 branch from acf24fc to af8954b Compare February 16, 2026 13:17
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 24ce52a to 69b0a3b Compare February 16, 2026 13:17
@m-kus
m-kus changed the base branch from graphite-base/461 to pr-split/06-paginated-incoming February 16, 2026 13:17
@m-kus
m-kus changed the base branch from pr-split/06-paginated-incoming to graphite-base/461 February 16, 2026 16:01
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 69b0a3b to 4e2451d Compare February 16, 2026 16:02
@m-kus
m-kus force-pushed the graphite-base/461 branch from af8954b to 0b5f70f Compare February 16, 2026 16:02
@m-kus
m-kus changed the base branch from graphite-base/461 to pr-split/06-paginated-incoming February 16, 2026 16:02
@m-kus
m-kus force-pushed the pr-split/06-paginated-incoming branch from 0b5f70f to 44013c2 Compare February 16, 2026 17:54
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 4e2451d to 2e5c7a4 Compare February 16, 2026 17:54
@m-kus
m-kus changed the base branch from pr-split/06-paginated-incoming to graphite-base/461 February 17, 2026 12:18
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 2e5c7a4 to 0c3a580 Compare February 17, 2026 12:22
@m-kus
m-kus force-pushed the graphite-base/461 branch from 44013c2 to 50712b5 Compare February 17, 2026 12:22
@m-kus
m-kus changed the base branch from graphite-base/461 to main February 17, 2026 12:22
@m-kus
m-kus force-pushed the pr-split/07-paginated-subchannels branch from 0c3a580 to 9a4cb36 Compare February 17, 2026 12:23
@m-kus
m-kus merged commit cdbb708 into main Feb 17, 2026
5 of 6 checks passed
@m-kus
m-kus deleted the pr-split/07-paginated-subchannels branch February 17, 2026 12:26
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