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

runtime api: cache test #5541

Closed
wants to merge 5 commits into from
Closed

Conversation

sandreim
Copy link
Contributor

@sandreim sandreim commented May 17, 2022

paritytech/polkadot-sdk#829
Added overseer back pressure by removing the internal waiting queue, so ToFs are relevant.

Changed MAX_PARALLEL_REQUESTS to see if deduplicating parallel requests improves cache efficiency.

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label May 17, 2022
@sandreim sandreim added 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 May 17, 2022
.spawn_blocking(API_REQUEST_TASK_NAME, Some("runtime-api"), req);
self.active_requests.push(recv);
}
/// Returns true if our active_requeusts queue is full.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

typo

Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
Signed-off-by: Andrei Sandu <andrei-mihail@parity.io>
@sandreim sandreim closed this May 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. 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
Development

Successfully merging this pull request may close these issues.

1 participant