Skip to content

introduce a light weight mode for tracking block progression#190

Merged
Chengxuan merged 12 commits intohyperledger:mainfrom
kaleido-io:introduce-lightweight-tracking
Apr 30, 2026
Merged

introduce a light weight mode for tracking block progression#190
Chengxuan merged 12 commits intohyperledger:mainfrom
kaleido-io:introduce-lightweight-tracking

Conversation

@Chengxuan
Copy link
Copy Markdown
Contributor

@Chengxuan Chengxuan commented Apr 21, 2026

for #191

Requires hyperledger/firefly-transaction-manager#159

EVM implementation of the headBlockNumber tracking mode. In this new mode, new block hashes event are still used as a shoulder tap event to trigger the chain head refresh using eth_blockNumber call, which will fetch the latest chain head. However, the logic no longer makes the heavy calls to fetch the detailed information of each block; therefore, it hugely reduces the amount of data fetched from the configured JSON-RPC endpoint.

The following eth method is no longer required when headBlockNumber mode is used for tracking block progression.

  • eth_getBlockByHash
  • eth_getBlockByNumber

Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
@Chengxuan Chengxuan requested a review from a team as a code owner April 21, 2026 13:11
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
@Chengxuan Chengxuan force-pushed the introduce-lightweight-tracking branch from 55b733f to c1af5cc Compare April 21, 2026 13:37
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Copy link
Copy Markdown
Contributor

@hosie hosie left a comment

Choose a reason for hiding this comment

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

LGTM with 2 comments

Comment thread README.md
Comment thread pkg/ethblocklistener/confirmation_reconciler.go Outdated
Comment thread internal/ethereum/config.go Outdated
Chengxuan and others added 5 commits April 27, 2026 08:54
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Co-authored-by: John Hosie <john.hosie@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
… introduce-lightweight-tracking

Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Copy link
Copy Markdown
Contributor

@peterbroadhurst peterbroadhurst left a comment

Choose a reason for hiding this comment

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

Marking approval, as @hosie and @Chengxuan discussed this and reached a joint reasoned proposal on the spelling of:

  • The setting is connector.chainTrackingMode, and is enabled on the the specific connector implementation (firefly-evmconnect in this case)
  • The two possible values are:
    • full
    • light

These values are described in code comments on the FFTM side:

https://github.com/kaleido-io/firefly-transaction-manager/blob/852459b5a29d758290410fb27617a66d6fd28cf6/pkg/ffcapi/api.go#L104-L107

Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
Signed-off-by: Chengxuan Xing <chengxuan.xing@kaleido.io>
@Chengxuan
Copy link
Copy Markdown
Contributor Author

Chengxuan commented Apr 30, 2026

Since 457132d contains only removals of existing contents in the Readme, and I don't have access to the Snyk logs yet. I'm going to merge the PR despite the Snyk failure

@Chengxuan Chengxuan merged commit 781a32b into hyperledger:main Apr 30, 2026
4 of 5 checks passed
@Chengxuan Chengxuan deleted the introduce-lightweight-tracking branch April 30, 2026 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants