Skip to content

kvindexer: enable parsing event values which exceed the max value of int64 #767

@jmalicevic

Description

@jmalicevic

Bug Report

Setup

CometBFT version (use cometbft version or git rev-parse --verify HEAD if installed from source):

0.34.x, 0.37.x, 0.38.x

Have you tried the latest version: yes/no
yes

What happened?

For parsing event values within the kvindexer we use parseInt which will not parse values bigger than the maximum value of int64. As pointed out in #726, there are many chains using big denoms and we might end up in a situation where their data is not indexed properly.

ParseInt was used only for range queries in Comet and Tendermint core versions prior to 0.34.27. v0.34.27+ uses ParseInt in many other places.

What did you expect to happen?

If an event of the form: sender.amount = 10000000000000000000 exists, a query for it should return this event. Currently, an error is produced.

Metadata

Metadata

Assignees

Labels

backlogA prioritized task in the team's backlogbugSomething isn't workingindexer

Type

No type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions