Skip to content

Implement Jetstream Support#40

Merged
rudyfraser merged 5 commits intoblacksky-algorithms:mainfrom
TheRipperoni:feature-jetstream
Jan 25, 2025
Merged

Implement Jetstream Support#40
rudyfraser merged 5 commits intoblacksky-algorithms:mainfrom
TheRipperoni:feature-jetstream

Conversation

@TheRipperoni
Copy link
Contributor

This creates an additional service that can be used as an alternative to the firehose subscriber. Advantage being significantly less bandwidth consumption as well as cpu time.

It also allows for customizing which lexicons you want to specifically query.

Copy link
Member

@rudyfraser rudyfraser left a comment

Choose a reason for hiding this comment

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

Couple small changes I'd like you to take a look at for main.rs but then otherwise should be good to go. Thanks for getting started on this!

@rudyfraser
Copy link
Member

Oh, one other thing. Because this is a jetstream-subscriber and not meant as a jetstream implementation could you rename the package as such? Wouldn't want there to be a namespace clash down the line if someone wanted to implement jetstream in the future for rsky.

@TheRipperoni
Copy link
Contributor Author

Oh, one other thing. Because this is a jetstream-subscriber and not meant as a jetstream implementation could you rename the package as such? Wouldn't want there to be a namespace clash down the line if someone wanted to implement jetstream in the future for rsky.

Updated to rename the package to rsky-jetstream-subscriber, as this make it much more clear what it does

@rudyfraser rudyfraser merged commit b3e9d6f into blacksky-algorithms:main Jan 25, 2025
3 of 4 checks passed
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