helium / blockchain-etl

Blockchain follower that follows and stores the Helium blockchain
Apache License 2.0
64 stars 37 forks source link

Only set failed status if current status is not cleared #161

Closed madninja closed 3 years ago

madninja commented 3 years ago

Since the txn_mgr will retry transactions across elections, the transaction could clear while the txn mgr is sending retries which would then come back as failures.

This would cause the pending_txn to be marked as failed after it had cleared.

This patch ensures that a failed status is only set if the current status is not already cleared