Jaspervdm - Progress update thread

Hi, here’s another update from me!

  • The review of grin#3453 is still in progress, but moving along well thanks to antioch. There are some subtleties in this PR, which is why review is being done carefully. I have made some adjustments based on his suggestions.
  • The RFC accompanying this PR has been updated and is nearing completion.
  • On the implementation of the new sync process that will be using the new p2p messages from the above RFC: I have been doing some experimenting with finding the most optimal method. Some technical details: right now all our incoming messages are handled by different “chain adapter” layers, which provide a good separation of concerns (e.g. keeping track of requested data and banning of peers is done by different layers). This does however mean that the chain is locked all over the place, which requires more coordination between each p2p reader and is less efficient. The alternative would be to create a channel that processes these messages in sequence, since writing to the chainstate is inherently sequential anyway. I am not entirely convinced either method is better, but am hoping to decide on it soon.
  • On the wallet side, me and yeastplume have been working together on a PR to provide a late locking mechanism. This will allow users to minimize the time their outputs stay locked, preventing the annoyance of locked outputs when the transacting counterparty is slow in responding with a slate.
9 Likes