Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Think about fast-tracking reverting TX #7

Open
ggwpez opened this issue Aug 18, 2021 · 3 comments
Open

Think about fast-tracking reverting TX #7

ggwpez opened this issue Aug 18, 2021 · 3 comments
Labels
proposal

Comments

@ggwpez
Copy link

ggwpez commented Aug 18, 2021

comment hyperledger-labs/go-perun#101 (comment)

Current situation:
TX will be deemed final only after being included in TxFinalityDepth many consecutive blocks after hyperledger-labs/go-perun#97 is done.

Thought:
While implementing this i thought about maybe fast-tracking reverting TX to not wait for TxFinalityDepth.
@ggwpez ggwpez mentioned this issue Aug 18, 2021
Merged
@matthiasgeihs
Copy link
Contributor

What is a reverting TX?

@ggwpez
Copy link
Author

ggwpez commented Aug 19, 2021

What is a reverting TX?

A TX that was rejected by the network for whatever reason.
Could be "out of gas" or "channel already registered".

I wasnt sure if we can optimize this case, but is is not really important to begin with i think since it rarely happens.

@matthiasgeihs
Copy link
Contributor

OK, now I get it.

@matthiasgeihs matthiasgeihs transferred this issue from hyperledger-labs/go-perun Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
proposal
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ggwpez @matthiasgeihs