Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

state/kvindexer: porting from 0.34 query fix #9876

Draft
wants to merge 3 commits into
base: v0.37.x
Choose a base branch
from
Draft

state/kvindexer: porting from 0.34 query fix #9876

wants to merge 3 commits into from

Conversation

jmalicevic
Copy link
Contributor

@jmalicevic jmalicevic commented Dec 14, 2022 •
edited by lasarojc

This is a forward port of #9759 to fix the querying behaviour where attributes should be returned only if they match within the same event.

PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

jmalicevic and others added 3 commits December 14, 2022 13:27
@jmalicevic @ancazamfir
@sergio-mena @romac @thanethomson
state/kvindexer: associate event attributes with events (#9759)
b6d99be 
* Updated event sequencing and added query keyword

* code cosmetics

* Documentation update

* Added per event indexing and querying to txindexer

* rpc test fix

* Added support for older versions where event sequencing is not supported

* Added support for old versions to tx indexer

* Added RPC match flag, fixed bugs in tx indexer, added tests

* Removed reference to match.events from the docs

* Openapi update

* Added height deduplication
Co-authored-by: Thane Thomson <connect@thanethomson.com>

Co-authored-by: Anca Zamfir <zamfiranca@gmail.com>
Co-authored-by: Sergio Mena <sergio@informal.systems>
Co-authored-by: Romain Ruetschi <romain.ruetschi@gmail.com>
Co-authored-by: Thane Thomson <connect@thanethomson.com>
@jmalicevic
Backport kvindexer fix
6fe2b13
@jmalicevic
Fixed match.events order in light client rpc
b9f5a5b
@jmalicevic jmalicevic added C:indexer Component: Indexer S:wip Work in progress (prevents stalebot from automatically closing) labels Dec 14, 2022
@jmalicevic jmalicevic self-assigned this Dec 14, 2022
@jmalicevic jmalicevic requested a review from a team December 14, 2022 12:39
@jmalicevic jmalicevic marked this pull request as draft December 15, 2022 10:35
@jmalicevic jmalicevic mentioned this pull request Jan 5, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ebuchman ebuchman Awaiting requested review from ebuchman

@adizere adizere Awaiting requested review from adizere

@lasarojc lasarojc Awaiting requested review from lasarojc

At least 1 approving review is required to merge this pull request.

Assignees

@jmalicevic jmalicevic

Labels
C:indexer Component: Indexer S:wip Work in progress (prevents stalebot from automatically closing)
Projects
Tendermint
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@jmalicevic