fix: Separate indexer setup #10032
Merged
fix: Separate indexer setup #10032
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Qwerty5Uiop
force-pushed
the
ap-indexer-api-split
branch
from
5086ed6
to
54c7ca9
Compare
vbaranov
reviewed
May 13, 2024
There was a problem hiding this comment.
finished_indexing?/1 works impropelry - it returns true, if indexer is disabled. So, it will always true in the setup with separated indexer and API pods.
There was a problem hiding this comment.
I suppose the same issue is true with indexed_ratio_internal_transactions/0.
There was a problem hiding this comment.
Processed in 57501f5
Qwerty5Uiop
force-pushed
the
ap-indexer-api-split
branch
from
9bd383a
to
57501f5
Compare
vbaranov
approved these changes
May 14, 2024
fedor-ivn
added a commit
that referenced
this pull request
May 18, 2024
* master: (65 commits) fix: Add healthcheck endpoints for indexer-only setup (#10076) 6.6.0 fix: Rework revert_reason (#9212) fix: Eliminate from_address_hash == #{address_hash} clause for transactions query in case of smart-contracts (#9469) feat: Add optional retry of NFT metadata fetch in Indexer.Fetcher.Tok… (#10036) fix: Separate indexer setup (#10032) feat: Blueprint contracts support (#10058) chore: Update hackney pool size: add new fetchers accounting (#9941) fix: Disallow batched queries in GraphQL endpoint (#10050) feat: Clone with immutable arguments proxy pattern (#10039) fix: vyper contracts re-verificaiton (#10053) refactor: Refactor get_additional_sources/4 -> get_additional_sources/3 (#10046) chore: Bump credo from 1.7.5 to 1.7.6 (#10060) chore: Bump redix from 1.5.0 to 1.5.1 (#10059) chore: Bump ex_doc from 0.32.1 to 0.32.2 (#10061) fix: Fix Unknown UID bug at smart-contract verification (#9986) refactor: test database config (#9662) chore: remove `has_methods` from `/addresses` (#10051) feat: Improve retry NFT fetcher (#10027) chore: Add support of Blast-specific L1 OP withdrawal events (#10049) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #9254
Resolves #8890
Changelog
Chain.indexed_ratio_blocks/0to use last block from DB to indicate if indexer is running instead ofDISABLE_INDEXERenv.DISABLE_WEBAPP=true.Explorer.Chain.Events.Listenerenabledlogic so it will be started in API setup.DISABLE_APIenv variable. If set,BlockScoutWebapplication won't start at all.INDEXER_INTERNAL_TRANSACTIONS_INDEXING_FINISHED_THRESHOLDtoAPI_INTERNAL_TRANSACTIONS_INDEXING_FINISHED_THRESHOLDsince it's used only for user requests.Docs update: blockscout/docs#278, blockscout/docs#279