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

full_node: More set usage in subscription code #15420

Conversation

xdustinface
Copy link
Contributor

Purpose:

Avoid potentially querying duplicated coin states from the DB by just filtering the condition items before and remove some list<>set conversions.

Current Behavior:

  1. Query based on a list of subscription items
  2. Create a set of CoinState items
  3. Convert the set to a list

New Behavior:

  1. Query based on a set of subscription items
  2. Create a list of CoinState items

Based on:

@xdustinface xdustinface added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Jun 1, 2023
@xdustinface xdustinface force-pushed the pr-full-node-more-set-usage-in-subscriptions branch from ddec6ea to 467f73b Compare June 7, 2023 15:57
@xdustinface xdustinface marked this pull request as ready for review June 7, 2023 19:49
@xdustinface xdustinface requested a review from a team as a code owner June 7, 2023 19:49
@wallentx wallentx merged commit e3beca2 into Chia-Network:main Jun 9, 2023
196 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants