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.

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

fix: fix query archival states before the upgrade #2390

Closed
wants to merge 1 commit into from

Conversation

catShaark
Copy link

@catShaark catShaark commented Apr 9, 2023

Closes: #2313

I use the sdk with modified CacheMultiStoreWithVersion to support querying archival states before the upgrade

Copy link
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

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

I would highly recommend against using a fork of the SDK.

@catShaark catShaark marked this pull request as draft April 10, 2023 08:05
@catShaark
Copy link
Author

@alexanderbez, sorry, I'll convert this to a draft PR

@faddat
Copy link
Contributor

faddat commented Apr 10, 2023

@alexanderbez so, I guess you mean use sdk 47?

eg: 45 has been eol'd and Khanh's pr to that has been closed because it is eol'd, and folks do want to make queries to older gaia.

@catShaark catShaark marked this pull request as ready for review April 10, 2023 19:18
@mpoke
Copy link
Contributor

mpoke commented Apr 11, 2023

@catShaark thanks for the contribution. Closing this PR though based on the following comment #2390 (review)

@mpoke mpoke closed this Apr 11, 2023
@faddat
Copy link
Contributor

faddat commented Apr 11, 2023

and how shall users get their queries on?

These changes were rejected on the 45 series sdk because it's been EOL'd, so there's a need to use a fork

@alexanderbez -- I agree with you we shouldn't be on a fork here....

@mpoke
Copy link
Contributor

mpoke commented Apr 14, 2023

and how shall users get their queries on?

@faddat please see #2313 (comment)

@faddat
Copy link
Contributor

faddat commented Apr 15, 2023

The hub is already on a fork of the SDK

1 similar comment
@faddat
Copy link
Contributor

faddat commented Apr 15, 2023

The hub is already on a fork of the SDK

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.

Unable to query archival data with recent upgrade 14470501
4 participants