Skip to content
This repository was archived by the owner on Apr 23, 2024. It is now read-only.

sdk/state: change horizon ingestion to only ingest txs for the escrow accounts #315

Closed
leighmcculloch opened this issue Sep 10, 2021 · 1 comment

Comments

@leighmcculloch
Copy link
Contributor

leighmcculloch commented Sep 10, 2021

When I implemented Horizon ingestion I initially made it stream transactions that affected the escrow accounts, but this resulted in out-of-order and duplicate transactions which was a problem with the state packages ingestion code at the time, so I changed the Horizon ingestion to simply stream all transactions on the whole network which ensured they would all be streamed in order.

With the changes @acharb is making for #267 (#312) and that we plan to make for #314, it will be possible to ingest transactions out of order and to ingest duplicates. Once those issues are closed we can change Horizon ingestion back to stream transactions from only the escrow accounts.

Dependent on #267 #314

@leighmcculloch
Copy link
Contributor Author

Duplicate of #320.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant