Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

refactor runtime cache for SessionInfo #4723

Merged
merged 2 commits into from
Jan 15, 2022
Merged

refactor runtime cache for SessionInfo #4723

merged 2 commits into from
Jan 15, 2022

Conversation

ordian
Copy link
Member

@ordian ordian commented Jan 15, 2022

The query invocation doesn't use the macro, but it's more explicit now.

Addresses #4706 (comment).

@ordian ordian added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jan 15, 2022
@ordian ordian requested a review from bkchr January 15, 2022 10:55
bkchr
bkchr approved these changes Jan 15, 2022
View reviewed changes
Copy link
Member

@bkchr bkchr left a comment

Choose a reason for hiding this comment

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

Ty!

@ordian
Merge branch 'master' into ao-4706-followup
4d500bd 
* master:
  spellcheck fixes (#4724)
  Report the actual missing session. (#4722)
@ordian
Copy link
Member Author

ordian commented Jan 15, 2022

bot merge

@paritytech-processbot paritytech-processbot bot merged commit eca33bf into master Jan 15, 2022
@paritytech-processbot paritytech-processbot bot deleted the ao-4706-followup branch January 15, 2022 20:10
Wizdave97 pushed a commit to ComposableFi/polkadot that referenced this pull request Feb 3, 2022
@ordian @Wizdave97
refactor runtime cache for SessionInfo (paritytech#4723)
d81babc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bkchr bkchr bkchr approved these changes

Assignees
No one assigned
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ordian @bkchr