Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

refactor function for stable perf #19386

Conversation

tao-stones
Copy link
Contributor

Problem

Avoiding push items into vector without capacity, which could unpredictably impact performance, depending on number of items being pushed into vector.

Summary of Changes

  1. refactored checking cost limit into a separate function, using iterator partition instead of pushing retryable transaction index into vector.
  2. added test cases.

Fixes #19275 (part of)

@stale
Copy link

stale bot commented Sep 4, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Sep 4, 2021
@stale
Copy link

stale bot commented Sep 21, 2021

This stale pull request has been automatically closed. Thank you for your contributions.

@stale stale bot closed this Sep 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transaction Cost updates
1 participant