Skip to content

all: use timer instead of time.After in loops, to avoid memleaks#91

Merged
FletcherMan merged 1 commit intomainfrom
ryan/time_after_leak
Jun 12, 2024
Merged

all: use timer instead of time.After in loops, to avoid memleaks#91
FletcherMan merged 1 commit intomainfrom
ryan/time_after_leak

Conversation

@ryanmorphl2
Copy link
Copy Markdown
Contributor

@ryanmorphl2 ryanmorphl2 requested a review from FletcherMan June 12, 2024 03:44
@ryanmorphl2 ryanmorphl2 requested a review from a team as a code owner June 12, 2024 03:44
@FletcherMan FletcherMan added the release/0.3.x indicates that needs to be cherry-pick to release/0.3.x branch label Jun 12, 2024
@FletcherMan FletcherMan merged commit b3fc060 into main Jun 12, 2024
@FletcherMan FletcherMan deleted the ryan/time_after_leak branch June 12, 2024 06:50
FletcherMan pushed a commit that referenced this pull request Jun 12, 2024
all: use timer instead of time.After in loops, to avoid memleaks (#91)

Co-authored-by: Ryan He <163962984+ryanmorphl2@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/0.3.x indicates that needs to be cherry-pick to release/0.3.x branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants