RFC 57: Safecoin Revised

Development RFCs
139

My remarks on this RFC:

  • Each CoinBalance has an associated queue named credits, but it isn’t stored in struct CoinBalance. So, where is it stored?

  • credits is a queue but no POP operations are described in the RFC. The Drawbacks section suggests that there is a size limit, but its value isn’t mentioned anywhere.

  • Relocating chunks seems risky because a fundamental invariant is not maintained anymore: the 8 nodes nearest to its address store a chunk (8 or whatever duplication factor is). I propose that elders kill a node that cannot a store a chunk, at least in a first version and improve it later.

  • Also remove the concept of good node and, instead, find a way to truly measure the space dedicated by a vault. There was a very good topic created by @mav on the subject (Proof Of Storage prototype).

4 Likes