search

osmosis-labs / osmosis

The AMM Laboratory
https://app.osmosis.zone
Apache License 2.0
880 stars 568 forks source link

feat/refactor(indexer): finalize indexer ingest refactor and publish pairs (backport #8487) #8488

Closed mergify[bot] closed 1 week ago

mergify[bot] commented 2 weeks ago

Closes: #XXX

What is the purpose of the change

Please read this Slack thread for a detailed explanation of the work accomplished here: https://osmosis-network.slack.com/archives/C060VCJAVBL/p1720223177058649

Relevant indexer PR: https://github.com/osmosis-labs/indexer/pull/9

Testing


curl "https://asia-southeast1-osmosis-data-team.cloudfunctions.net/roman-pair?id=1066-0-1" | jq .
{
  "pair": {
    "id": "1066-0-1",
    "dexKey": "osmosis",
    "asset0Id": "ibc/0CD3A0285E1341859B5E86B6AB7682F023D03E97607CCC1DC95706411D866DF7",
    "asset1Id": "uosmo",
    "feeBps": 30
  }
}
```<hr>This is an automatic backport of pull request #8487 done by [Mergify](https://mergify.com).
mergify[bot] commented 2 weeks ago

Cherry-pick of 3b67de25a8cbaed5da5a15de1ff56f2c0e30e399 has failed:

On branch mergify/bp/v25.x/pr-8487
Your branch is up to date with 'origin/v25.x'.

You are currently cherry-picking commit 3b67de25.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
    modified:   cmd/osmosisd/cmd/root.go
    deleted:    ingest/indexer/domain/cold_start_manager.go
    modified:   ingest/indexer/domain/errors.go
    modified:   ingest/indexer/domain/keepers.go
    new file:   ingest/indexer/domain/mocks/publisher_mock.go
    new file:   ingest/indexer/domain/pair.go
    modified:   ingest/indexer/domain/pool.go
    modified:   ingest/indexer/domain/publisher.go
    modified:   ingest/indexer/indexer_config.go
    modified:   ingest/indexer/publisher.go
    new file:   ingest/indexer/service/blockprocessor/block_process_indexer_factory.go
    new file:   ingest/indexer/service/blockprocessor/block_process_indexer_factory_test.go
    new file:   ingest/indexer/service/blockprocessor/block_updates_indexer_block_process_strategy.go
    new file:   ingest/indexer/service/blockprocessor/full_indexer_block_process_strategy.go
    new file:   ingest/indexer/service/blockprocessor/pair_publisher.go
    modified:   ingest/indexer/service/writelistener/bank_write_listener_test.go
    modified:   ingest/sqs/service/blockprocessor/block_updates_sqs_block_process_strategy_test.go
    modified:   ingest/sqs/service/blockprocessor/full_sqs_block_process_strategy_test.go

Unmerged paths:
  (use "git add <file>..." to mark resolution)
    both modified:   app/app.go
    both modified:   ingest/indexer/service/client/pubsub_client.go
    both modified:   ingest/indexer/service/indexer_streaming_service.go
    both modified:   ingest/indexer/service/writelistener/bank_write_listener.go

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

PaddyMc commented 1 week ago

Just FYI this PR will fix the backport => https://github.com/osmosis-labs/osmosis/pull/8492