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

[Feature]: Improve implementation of get_state_sync removing details #309

Closed
phklive opened this issue Apr 10, 2024 · 3 comments
Closed
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@phklive
Copy link
Contributor

phklive commented Apr 10, 2024

To whom is this feature for?

End user

Why is is this feature needed?

To reduce the amount of computation made by get_state_sync

pub fn get_state_sync(

How is this feature used?

As of now the sql queries done on the store database for get_state_sync requires us to make another query with is select_notes_since_block_by_tag_and_sender to get notes.

We want to remove details from this query to be able to not have to make a high amount of work. If needed the user will make a subsequent query to get_notes_by_id to query on-chain note details.

Anything else?

#300 (comment)

@phklive phklive added the enhancement New feature or request label Apr 10, 2024
@bobbinth bobbinth added this to the v0.3 milestone Apr 12, 2024
@bobbinth bobbinth modified the milestones: v0.3, v0.4 Apr 29, 2024
@Dominik1999
Copy link
Contributor

Would a user be able to get a note by its ID? The issue is not super clear to me

@bobbinth
Copy link
Contributor

The user should already be able to do that. This issue is of an optimization issue for how things are handled internally (currently, we do more work than needed in some situations).

@bobbinth bobbinth modified the milestones: v0.4, v0.5 Jun 10, 2024
@bobbinth bobbinth modified the milestones: v0.5, v0.6 Aug 18, 2024
@Dominik1999 Dominik1999 moved this to In Progress in User's testnet Aug 26, 2024
@polydez polydez moved this from In Progress to In Review in User's testnet Aug 27, 2024
@polydez
Copy link
Contributor

polydez commented Aug 30, 2024

Resolved by #462

@polydez polydez closed this as completed Aug 30, 2024
@github-project-automation github-project-automation bot moved this from In Review to Done in User's testnet Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

4 participants