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

rpc: Implement GetUnspentTransactionOutputSet #1145

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

stevenroose
Copy link
Contributor

@stevenroose stevenroose commented Mar 13, 2018

Requires #1075. Will also be simpler with #1045.

Implements a new asynchronous method to retrieve the entire UTXO set for the best chain (at the moment of calling).

I'm not entirely happy with the way the hash of the chain tip of the moment the UTXO snapshot is taken is communicated. Because gRPC allows methods to return streams, the one value could not be returned in a call. Instead, it is sent by the caller (who sends the best hash at the moment of calling) and the server returns an error if a new block was added to the blockchain in the meantime.

To avoid confusion when introducing gRPC support.
@stevenroose stevenroose mentioned this pull request Mar 14, 2018
@stevenroose stevenroose changed the title GRPC: Implement GetUnspentTransactionOutputSet rpc: Implement GetUnspentTransactionOutputSet Apr 8, 2018
@stevenroose
Copy link
Contributor Author

Idea: ScanUnspentOutputSet. Cfr the way Core implemented scantxoutset.

@ajjaah
Copy link

ajjaah commented Sep 25, 2018

Hello there! Can anyone give the update on this feature please? It would be very useful to have this rpc call. Could this be implemented or are there any blocking issues to make it work? Thanks!

@stevenroose
Copy link
Contributor Author

Yeah this is based on the gRPC implementation that I worked on in the past. I'm not doing much work on btcd anymore, but the gRPC impl is quite finished tbh. Whether it will ever land is the question :)

I'd be very happy to guide you through it if you would decide to pick it up.

@jakesylvestre
Copy link
Collaborator

@jcvernaleo (as per #1530)

  • Low priority
  • Enhancement
  • Outdated (very)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants