Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FIX: [core] fix memory leak #1728

Merged
merged 1 commit into from
Sep 5, 2024
Merged

FIX: [core] fix memory leak #1728

merged 1 commit into from
Sep 5, 2024

Conversation

kbearXD
Copy link
Collaborator

@kbearXD kbearXD commented Sep 3, 2024

No description provided.

@bbgokarma-bot
Copy link

Welcome back! @kbearXD, This pull request may get 213 BBG.

c9s
c9s reviewed Sep 4, 2024
View reviewed changes
pkg/bbgo/marketdatastore.go Outdated Show resolved Hide resolved
c9s
c9s reviewed Sep 4, 2024
View reviewed changes
pkg/bbgo/marketdatastore.go Outdated Show resolved Hide resolved
c9s
c9s reviewed Sep 4, 2024
View reviewed changes
pkg/bbgo/marketdatastore.go Outdated Show resolved Hide resolved
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 247 BBG

@kbearXD kbearXD requested a review from c9s September 4, 2024 08:50
@c9s c9s changed the title FIX: fix memory leak FIX: [core] fix memory leak Sep 4, 2024
c9s
c9s reviewed Sep 4, 2024
View reviewed changes
pkg/bbgo/marketdatastore.go Outdated Show resolved Hide resolved
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 324 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 328 BBG

c9s
c9s reviewed Sep 5, 2024
View reviewed changes
pkg/bbgo/marketdatastore.go Outdated Show resolved Hide resolved
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 343 BBG

@kbearXD kbearXD merged commit f2a443a into main Sep 5, 2024
3 checks passed
@kbearXD kbearXD deleted the kbearXD/dca2/fix-memory-leak branch September 5, 2024 09:52
@bbgokarma-bot
Copy link

Hi @kbearXD,

Well done! 348 BBG has been sent to your polygon wallet. Please check the following tx:

https://polygonscan.com/tx/0x9e30fb5a2fdd7f50f932cfdcba519bb8d6099cbf9af6c2bfc6c7b87da3ebeb47

Thank you for your contribution!

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

@c9s c9s c9s approved these changes

@ycdesu ycdesu Awaiting requested review from ycdesu

@gx578007 gx578007 Awaiting requested review from gx578007

@bailantaotao bailantaotao Awaiting requested review from bailantaotao

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kbearXD @bbgokarma-bot @c9s