On 28.02.2012 23:06, Luke-Jr wrote: > Please review and comment/critique: > https://en.bitcoin.it/wiki/BIP_DRAFT:_getmemorypool
Looking forward to implementing this in my pool backend and miner. A few comments: "transactions add or remove transactions (both of the above; default if "transactions" omitted) " In the above, you may want to specify that the "transactions" referred to here is the one in the first table (JSON-RPC response object) and not the mutations. For a moment I thought free tx editing was the default. Long polling as currently implemented in pools has a race condition. Does the miner reconnect first or does another block change happen first? "Double" block changes are common with merged mining and I'm doing all sorts of tricks in my pool backend to reduce this problem. How about another entry "longpollid" in long poll responses. The last seen longpollid should be included by the client in future long poll requests. This enables the server to see if the client has missed any block changes. The ID could perhaps be submitted in an HTTP header (X-LongPollID?) if we wish to keep the JSON-RPC params empty, or params could hold an object with a key "longpollid". Could be a string or number, like "workid". Another useful value in the getmemorypool response would be "height", so the miner can include the correct height in the coinbase. I would like that in bitcoind as well. One JSON-RPC call instead of two, and no race condition between getmemorypool and getblocknumber. It should be explained how target vs. fulltarget works. Perhaps some things should be optional for a client to implement? I think "noncerange" is of limited use and there's a good chance of getting the endianness wrong. Regards, Geir Harald Hansen ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Bitcoin-development mailing list Bitcoin-development@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bitcoin-development
