-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use QueryTip for local state queries #1053
Conversation
Transactions CostsSizes and execution budgets for Hydra protocol transactions. Note that unlisted parameters are currently using
Script summary
Cost of Init Transaction
Cost of Commit TransactionThis is using ada-only outputs for better comparability.
Cost of CollectCom Transaction
Cost of Close Transaction
Cost of Contest Transaction
Cost of Abort TransactionSome variation because of random mixture of still initial and already committed outputs.
Cost of FanOut TransactionInvolves spending head output and burning head tokens. Uses ada-only UTxO for better comparability.
|
48af485
to
e057b44
Compare
dedf2fb
to
5c8168d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm really looking forward to find out which query is failing via the logs!
When we know, I think we can solve this at the root and change the query correctly to be not prone to some race condition!?
Marking this as request changes because we can't merge it like this because of the tracing though.
if n == (0 :: Int) | ||
then throwIO $ QueryAcquireException err | ||
else do | ||
tracer ("Retrying query... " ++ show n <> " Query: " <> show query) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will make our logs non-JSON and hence it can only be temporary / blocks merging of this IMO.
So far using query at the tip works. Let's wait one more day and say this is a fix if we don't see any Retrying query logs. |
cd296da
to
198f3e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please rename the PR title? Change looks good!
fix #560
Note: Retry attempts should probably come from a configuration value.