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

NDEV-2176: Change the ordering of data in neon_revision #189

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

Deniskore
Copy link

@Deniskore Deniskore commented Sep 8, 2023

We must consistently provide the lower neon_revision version. Therefore, if we possess neon_revision for slots 1000 and 2000, and a query requests slot 1700, we should give the neon_revision version associated with the 1000 slot.

@Deniskore Deniskore force-pushed the NDEV_2176_fix_sysvarrent branch from f6a93fc to d2d8983 Compare September 8, 2023 13:50
@Deniskore Deniskore changed the title NDEV-2176: Fix query for SysvarRent, change the ordering in neon_revi… NDEV-2176: Change the ordering of data in neon_revision Sep 8, 2023
@Deniskore Deniskore force-pushed the NDEV_2176_fix_sysvarrent branch from d2d8983 to 843627c Compare September 8, 2023 15:25
stanislav-tkach
stanislav-tkach previously approved these changes Sep 8, 2023
@andreisilviudragnea andreisilviudragnea dismissed stale reviews from stanislav-tkach and themself via d2d8983 September 11, 2023 08:49
@andreisilviudragnea andreisilviudragnea force-pushed the NDEV_2176_fix_sysvarrent branch 2 times, most recently from d2d8983 to 843627c Compare September 11, 2023 08:51
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