Skip to content

Conversation

injectives
Copy link
Contributor

@injectives injectives commented Jun 26, 2023

This update introduces a new TransactionTerminatedException that is a subclass of the ClientException. The server Neo.ClientError.Transaction.Terminated error code maps to this exception now, making it more fine-grained.

In addition, this update brings a PRIVATE API for transaction termination. It replaces the InternalReactiveTransaction.interrupt() and provides a similar functionality that is also available in the synchronous API. Please note that this is not a public API.

@injectives injectives marked this pull request as draft June 26, 2023 20:44
@injectives injectives force-pushed the feature/txt branch 11 times, most recently from 0472f1c to 65d7da5 Compare June 27, 2023 10:22
@injectives injectives changed the title [INTERNAL] Add tx t [INTERNAL] Add transaction termination Jun 27, 2023
@injectives injectives marked this pull request as ready for review June 27, 2023 10:39
@injectives injectives requested a review from gjmwoods June 27, 2023 11:51
@injectives injectives force-pushed the feature/txt branch 2 times, most recently from 77d34bd to aef1812 Compare June 27, 2023 16:31
@injectives injectives changed the title [INTERNAL] Add transaction termination Introduce TransactionTerminatedException Jun 27, 2023
This update introduces a new `TransactionTerminatedException` that is a subclass of the `ClientException`. The server `Neo.ClientError.Transaction.Terminated` error code maps to this exception now, making it more fine-grained.

In addition, this update brings a PRIVATE API for transaction termination. It replaces the `InternalReactiveTransaction.interrupt()` and provides a similar functionality that is also available in the synchronous API. Please note that this is not a public API.
@injectives
Copy link
Contributor Author

Closed in favour of #1445.

@injectives injectives closed this Jul 5, 2023
@injectives injectives deleted the feature/txt branch July 5, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant