Roman Puchkovskiy created IGNITE-18499:
------------------------------------------
Summary: Make IncomingSnapshotCopier behave closer to
LocalSnapshotCopier
Key: IGNITE-18499
URL: https://issues.apache.org/jira/browse/IGNITE-18499
Project: Ignite
Issue Type: Improvement
Reporter: Roman Puchkovskiy
Fix For: 3.0.0-beta2
There are a few differences between {{IncomingSnapshotCopier}} and
{{{}LocalSnapshotCopier{}}}. JRaft seems to be very sensitive to little changes
in behavior because it relies on many invariants, some of them implicit, to
function correctly. Hence it is best to have our copier mimic the stock one as
fully as possible (where it's feasible).
Here are the differences:
# On cancellation, the copier should be transferred to an erroneous state
(using {{{}setError(){}}})
# On closure, the copier should be cancelled
# The stock copier does not invoke {{join()}} on cancellation, but our
implementation does. On the one hand, waiting seems to make sense because it
allows to save retries, but it's a difference from the stock logic which might
be dangerous.
# We should only set error if it has not been set yet; but our copier
implementation usually does the opposite: if there is an error, it sets some
other error
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
