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.

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

Ocean: fix rm/update oracle index #3107

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

canonbrother
Copy link
Contributor

@canonbrother canonbrother commented Nov 1, 2024

Summary

  • fix passing wrong oracle id to indexer(RemoveOracle & UpdateOracle)
  • better NotFoundIndex message
-- let oracle_id = ctx.tx.id;
++ let oracle_id = self.oracle_id;

@canonbrother canonbrother force-pushed the canonbrother/fix-rm-oracle-index branch from 82cd86b to 434a558 Compare November 1, 2024 07:12
@canonbrother canonbrother changed the title Ocean: fix rm oracle index Ocean: fix rm/update oracle index Nov 4, 2024
@canonbrother canonbrother force-pushed the canonbrother/fix-rm-oracle-index branch from 434a558 to 0887c5f Compare November 4, 2024 05:21
@canonbrother canonbrother force-pushed the canonbrother/fix-rm-oracle-index branch 2 times, most recently from 1206e36 to 363ed1a Compare November 4, 2024 14:41
@@ -184,7 +184,7 @@ fn create_new_bucket(

impl IndexBlockStart for PoolSwap {
fn index_block_start(self, services: &Arc<Services>, block: &BlockContext) -> Result<()> {
let mut pool_pairs = services.pool_pair_cache.get();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert as breaking poolpair.swap.split.defid.ts test

@canonbrother canonbrother marked this pull request as ready for review November 4, 2024 15:10
@canonbrother canonbrother self-assigned this Nov 4, 2024
@Jouzo Jouzo merged commit 1b527c3 into ocean-catchup-on-startup Nov 4, 2024
5 of 6 checks passed
@Jouzo Jouzo deleted the canonbrother/fix-rm-oracle-index branch November 4, 2024 15:20
Bushstar pushed a commit that referenced this pull request Nov 5, 2024
* Ocean tries to catchup on startup if below tip height

* Keep hold of lock

* Fix RemoveOracle indexing

* Small perf improvements

* Revert "Keep hold of lock"

This reverts commit 07552f7.

* Create base txid out of loop

* Fix printf args order

* Don't hold ocean invalidation

* Revert "Revert "Keep hold of lock""

This reverts commit 29a423c.

* Bump setgovheight dftx buffer size

* Ocean: fix rm/update oracle index (#3107)

* fix

* update ocean ci

* revert to direct ffi:get_pp

* Remove clone

---------

Co-authored-by: canonbrother <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants