Skip to content

stats(native): pool per request#4118

Merged
istio-testing merged 1 commit intoistio:masterfrom
kyessenov:native_stats_debug
Oct 18, 2022
Merged

stats(native): pool per request#4118
istio-testing merged 1 commit intoistio:masterfrom
kyessenov:native_stats_debug

Conversation

@kyessenov
Copy link
Copy Markdown
Contributor

Signed-off-by: Kuat Yessenov kuat@google.com

DynamicPool is not working when shared between requests (heap-buffer-overrun). Make it per request and remove map lookups.

Signed-off-by: Kuat Yessenov <kuat@google.com>
@kyessenov kyessenov requested a review from a team October 18, 2022 04:07
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 18, 2022
@kyessenov
Copy link
Copy Markdown
Contributor Author

/retest

@istio-testing istio-testing merged commit 1bcaa93 into istio:master Oct 18, 2022
@kyessenov kyessenov added the cherrypick/release-1.16 Set this label on a PR to auto-merge it to the release-1.16 branch label Oct 20, 2022
@kyessenov kyessenov deleted the native_stats_debug branch October 20, 2022 03:11
@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: new pull request created: #4126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherrypick/release-1.16 Set this label on a PR to auto-merge it to the release-1.16 branch size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants