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.

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

feat: decode with coreDiscoveryId as well as coreDiscoveryKey #267

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gmaclennan
Copy link
Member

@gmaclennan gmaclennan commented Sep 24, 2024

Allow decode to work when a coreDiscoveryId is
passed, instead of coreDiscoveryKey to match an
upstream change in the multi-core-indexer to use
coreDiscoveryId rather than coreDiscoveryKey. This
reduces a round-trip conversion from buffer to
string, and will also facilitate sending entries
between threads when we run indexing in a separate
worker thread.

@gmaclennan gmaclennan self-assigned this Sep 24, 2024
@gmaclennan gmaclennan force-pushed the feat/core-discovery-id branch from 331e6f5 to ab4547b Compare September 24, 2024 11:15
Allow decode to work when a coreDiscoveryId is
passed, instead of coreDiscoveryKey to match an
upstream change in the multi-core-indexer to use
coreDiscoveryId rather than coreDiscoveryKey. This
reduces a round-trip conversion from buffer to
string, and will also facilitate sending entries
between threads when we run indexing in a separate
worker thread.
@gmaclennan gmaclennan force-pushed the feat/core-discovery-id branch from ab4547b to 65113ab Compare September 24, 2024 11:17
@gmaclennan gmaclennan changed the title feat!: decode with coreDiscoveryId rather than coreDiscoveryKey feat: decode with coreDiscoveryId as well as coreDiscoveryKey Sep 24, 2024
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.

1 participant