Skip to content

Commit

Permalink
Fixed the reindex runtime issue caused by missing API dependencies du…
Browse files Browse the repository at this point in the history
…ring reindexing (#143)
  • Loading branch information
yoozo authored Dec 6, 2024
1 parent 93f7244 commit 1bad0fc
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
3 changes: 3 additions & 0 deletions packages/node/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed
- Missing API dependency when using reindex

## [3.15.0] - 2024-11-26
### Added
- Support for historical indexing by timestamp as well as block height
Expand Down
8 changes: 6 additions & 2 deletions packages/node/src/subcommands/reindex.module.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ import { Module } from '@nestjs/common';
import { EventEmitter2, EventEmitterModule } from '@nestjs/event-emitter';
import { SchedulerRegistry } from '@nestjs/schedule';
import {
ConnectionPoolService,
ConnectionPoolStateManager,
DbModule,
ForceCleanService,
NodeConfig,
Expand Down Expand Up @@ -32,10 +34,12 @@ import { DynamicDsService } from '../indexer/dynamic-ds.service';
DynamicDsService,
PoiService,
DsProcessorService,
ConnectionPoolStateManager,
ConnectionPoolService,
{
// Used to work with DI for unfinalizedBlocksService but not used with reindex
provide: AlgorandApiService,
useFactory: () => undefined,
useFactory: AlgorandApiService.init,
inject: ['ISubqueryProject', ConnectionPoolService, EventEmitter2],
},
SchedulerRegistry,
],
Expand Down

0 comments on commit 1bad0fc

Please sign in to comment.