2018-07-24 04:26:26
[snowblossomcoin/snowblossom] Issue closed by fireduck64
GitHub
2018-07-24 04:27:00
[snowblossomcoin/snowblossom] Issue closed by fireduck64
GitHub
2018-07-24 04:31:57
*https://github.com/snowblossomcoin/snowblossom/compare/058e11c75f8d...d997d5c381f4*
https://github.com/snowblossomcoin/snowblossom/commit/d997d5c381f4a756ddb4fa87f485eb9bea8b730d - Switch to rocksdb 5.14.2.
GitHub
2018-07-24 04:31:57
[snowblossomcoin/snowblossom] Issue closed by fireduck64
GitHub
2018-07-24 04:38:22
@Fireduck nice, fixed the rocks db error?
I believe that was our most common.
Clueless
2018-07-24 04:38:37
[snowblossomcoin/snowblossom] Issue closed by fireduck64
GitHub
2018-07-24 04:38:59
@Clueless it was just a matter of wait for facebook to fix it
Fireduck
2018-07-24 04:39:04
they were aware of it
Fireduck
2018-07-24 04:39:21
it was some deep weird c++ compile issue with rocks db library
Fireduck
2018-07-24 04:39:42
have you seen the new jsonrpc on MrPlow?
Fireduck
2018-07-24 04:39:44
it is pretty sweet
Fireduck
2018-07-24 04:46:01
wrting params for jsonrpc send
Fireduck
2018-07-24 04:46:09
what should you call the address your are sending to?
Fireduck
2018-07-24 04:46:17
recipient is most correct, but i hate spelling that
Fireduck
2018-07-24 04:50:57
To from?
Clueless
2018-07-24 04:51:19
from_address
to_address
Clueless
2018-07-24 04:51:30
from address isn't something you can control in this case
Fireduck
2018-07-24 04:51:33
at least not right now
Fireduck
2018-07-24 04:51:46
input/output
Clueless
2018-07-24 04:53:05
{
"jsonrpc": "2.0",
"method": "send",
"id": "send_test",
"params": {
"broadcast": true,
"to": {
"snowtest:j4hl7axx24yy9m3pfmag3a3jn26f20rwz3u9jhr9": 25,
"snowtest:u4wmy9c8n56vd9285srvd4928ewm0urru2g7va2m": 50
}
}
}
Fireduck
2018-07-24 05:03:18
oh god, there are insane race conditions
Fireduck
2018-07-24 05:03:40
with the jsonrpc have to be ready for multiple things to happen at once
Fireduck
2018-07-24 05:03:46
which is fine
Fireduck
2018-07-24 05:04:03
but went building sends I need to make sure they don't use the same outputs
Fireduck
2018-07-24 05:04:35
the real solution is to have a pool of them in memory and let threads pull from them but that is a pain
Fireduck
2018-07-24 05:07:20
ok, I'll have something lock things while it is doing it
Fireduck
2018-07-24 05:07:35
which effectively serializes the operation, but it is proper
Fireduck
2018-07-24 05:07:42
and doing it better can be a later improvement
Fireduck
2018-07-24 05:48:49
*https://github.com/snowblossomcoin/snowblossom/compare/d997d5c381f4...5747394c470b*
https://github.com/snowblossomcoin/snowblossom/commit/5747394c470bfc5466c807b64d9e0ee5bded24a5 - jsonrpc send
GitHub
2018-07-24 08:47:30
java should have queue implementations built in?
Rotonen
2018-07-24 13:50:02
it has many
Fireduck
2018-07-24 14:02:21
great work on the RPC :thumbsup:
Shoots
2018-07-24 14:05:48
@Rotonen the problem isn't the queue, it is keeping it consistent as transactions are flowing. Which is doable, but takes some thought to get right.
Fireduck
2018-07-24 14:06:23
Right now I have it synchronized, so it is basically only building a single tx at a time
Fireduck
2018-07-24 14:06:38
which is probably fine for now. Good to be safe and proper before fast.
Fireduck
2018-07-24 14:06:42
Can make it fast later.
Fireduck
2018-07-24 16:59:17
iirc java had fancy concurrent queue stuff for ~banking transaction queue processing
Rotonen
2018-07-24 17:02:39
absolutely. The concurrent queue is the easy part.
Fireduck
2018-07-24 17:03:14
The consistency I was talking about is making sure the spendable transaction outputs in the queue are still spendable
Fireduck
2018-07-24 17:03:39
since other transactions could be spending them from other clients, we could have block reorgs, etc
Fireduck