Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(sdk): NET-1252 query operators on stream #2703

Merged
merged 16 commits into from
Aug 9, 2024
Merged

feat(sdk): NET-1252 query operators on stream #2703

merged 16 commits into from
Aug 9, 2024

Conversation

juslesan
Copy link
Contributor

@juslesan juslesan commented Aug 2, 2024 •
edited
Loading

Summary

Query recently onli operators on a stream from the graph.

Future improvements

  • Use the results of the query to make RPC calls to operators to discover nodes on partitions.
  • Better testing for OperatorRegistry
  • better configurability?

@linear Linear
Copy link

linear bot commented Aug 2, 2024

NET-1252 Accelerate stream entrypoint discovery by using Operator layer

@github-actions github-actions bot added the sdk label Aug 2, 2024
Base automatically changed from NET-1252-broker to main August 7, 2024 09:22
@juslesan juslesan marked this pull request as ready for review August 7, 2024 12:48
@juslesan juslesan requested a review from harbu August 7, 2024 12:48
harbu
harbu reviewed Aug 8, 2024
View reviewed changes
packages/sdk/src/StreamrClient.ts Outdated Show resolved Hide resolved
packages/sdk/src/contracts/OperatorRegistry.ts Outdated Show resolved Hide resolved
@juslesan juslesan requested a review from harbu August 8, 2024 13:03
@juslesan juslesan merged commit 54ada69 into main Aug 9, 2024
24 checks passed
@juslesan juslesan deleted the NET-1252-query-operators-on-stream branch August 9, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harbu harbu harbu approved these changes

Assignees
No one assigned
Labels
sdk
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@juslesan @harbu