-
Notifications
You must be signed in to change notification settings - Fork 1.7k
finer time resolution please #9468
Comments
Great, thanks a lot. So, is it automatically enabled, or do I have to use a switch? Which one? Thanks. |
@drandreaskrueger try this: "engine": {
"instantSeal": {
"params": {
"millisecondTimestamp": true
}
}
}, |
Can I use it as a commandline switch too?
? |
nope
|
Thanks, @ordian And @Tbaut has added it to his repo, see https://github.com/paritytech/wiki/pull/240 But please reopen. Now I have tried to adapt this file here: https://github.com/paritytech/parity-deploy/blob/master/config/spec/engine/instantseal into
but when I then re-run, with these commands
I still get whole seconds of time resolution, not milliseconds (i.e. 1000 ms or 0 ms, never anything else):
simply because
What are we doing wrong? Thanks a lot! P.S.: I am using this environment & benchmarking tool. |
"millisecondTimestamp" is available since 2.1.0 |
:-) |
I see. then I simply have to wait, until they have fixed this issue #9582 because the 2.x.y versions are not working well yet. |
I suggest you make this the default behavior of the instantseal chain. Because otherwise you end up with tons of blocks which have an identical block time (i.e. the same second). Weird side effects. |
If two blocks are closer in time than (half) a second, parity returns identical
web3.eth.getBlock(blockNumber).timestamp
for both blocks ....... which can then lead to ZeroDivisionError like here.
Of course, I can catch that, as you see. But ... but.
(For the context, please have a look at these chainhammer benchmarks --> parity.md)
--> Please increase the precision of the blocktime, thanks.
(
geth quorum raft consensus
is even using nanoseconds which is IMHO too extreme.)The text was updated successfully, but these errors were encountered: