Skip to content

feat: decode with coreDiscoveryId as well as coreDiscoveryKey#267

Draft
gmaclennan wants to merge 1 commit intomainfrom
feat/core-discovery-id
Draft

feat: decode with coreDiscoveryId as well as coreDiscoveryKey#267
gmaclennan wants to merge 1 commit intomainfrom
feat/core-discovery-id

Conversation

@gmaclennan
Copy link
Copy Markdown
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