Skip to content
This repository was archived by the owner on Sep 15, 2026. It is now read-only.

296 fix blocks too big - #292

Merged
nischitpra merged 2 commits into
np/committer_streamingfrom
np/296_fix
Sep 26, 2025
Merged

nischitpra merged 2 commits into
np/committer_streamingfrom
np/296_fix

Conversation

@nischitpra

@nischitpra nischitpra commented Sep 26, 2025 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features

    • Added CLI commands: backfill and committer.
    • Environment-based configuration via .env and environment variables.
    • Exposes Prometheus metrics on port 2112 with expanded per-project/per-chain metrics.
  • Improvements

    • Backfill writes Parquet files to S3 with dynamic batching and resiliency.
    • Committer streams processed data from S3 to downstream systems with memory-safe flow control.
    • Enhanced RPC handling for specific chains to improve block/transaction retrieval reliability.
  • Chores

    • Ignore Parquet files in version control.
    • Updated dependencies and docker-compose Kafka settings.

@zeet-co

zeet-co Bot commented Sep 26, 2025

Copy link
Copy Markdown

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://insight-296-hedera-testnet-backfi-d61367.insight.zeet.app before merging 😉

@coderabbitai

coderabbitai Bot commented Sep 26, 2025 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch np/296_fix

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between f66f782 and ef7e268.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (30)
  • .gitignore (1 hunks)
  • cmd/backfill.go (1 hunks)
  • cmd/committer.go (1 hunks)
  • cmd/migrate_valid.go (7 hunks)
  • cmd/root.go (1 hunks)
  • configs/config.go (3 hunks)
  • docker-compose.yml (0 hunks)
  • go.mod (2 hunks)
  • internal/backfill/backfill.go (1 hunks)
  • internal/backfill/disableIndexerMaybeStartCommitter.go (1 hunks)
  • internal/backfill/getbackfillboundaries.go (1 hunks)
  • internal/backfill/parquetwriter.go (1 hunks)
  • internal/committer/blockparserroutine.go (1 hunks)
  • internal/committer/blockprocessorroutine.go (1 hunks)
  • internal/committer/committer.go (1 hunks)
  • internal/committer/parquet.go (1 hunks)
  • internal/committer/poollatest.go (1 hunks)
  • internal/committer/semaphore.go (1 hunks)
  • internal/libs/clickhouse.go (1 hunks)
  • internal/libs/constants.go (1 hunks)
  • internal/libs/kafka.go (1 hunks)
  • internal/libs/libblockdata/getblockdata.go (1 hunks)
  • internal/libs/libblockdata/validator.go (1 hunks)
  • internal/libs/rpcclient.go (1 hunks)
  • internal/libs/s3.go (1 hunks)
  • internal/metrics/metrics.go (1 hunks)
  • internal/rpc/rpc.go (3 hunks)
  • internal/storage/kafka.go (2 hunks)
  • internal/storage/kafka_publisher.go (4 hunks)
  • internal/types/types.go (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgithub.com/​caarlos0/​env@​v3.5.0+incompatible91100100100100
Addedgithub.com/​joho/​godotenv@​v1.5.199100100100100

View full report

@nischitpra
nischitpra changed the base branch from main to np/committer_streaming September 26, 2025 19:57
@nischitpra
nischitpra merged commit dfb8bef into np/committer_streaming Sep 26, 2025
4 of 6 checks passed
@nischitpra
nischitpra deleted the np/296_fix branch September 26, 2025 19:57

This branch is being deployed

1 in progress deployment
production — ef7e2686 Deployed Sep 26, 2025 by zeet-co[bot]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant