Skip to content

[FIX] Implement actual ring buffer to prevent memory leaks when influ… #26

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[FIX] Implement actual ring buffer to prevent memory leaks when influ… #26

wants to merge 1 commit into from

Conversation

JanekFiranek
Copy link

@JanekFiranek JanekFiranek commented Mar 13, 2025 •
edited
Loading

.

@JanekFiranek JanekFiranek self-assigned this Mar 13, 2025
@Only1Loatheb
Copy link

I don't know if this change will actually help in case of influx unavaiability.
I think we should decrease unsent-buffer-size from 1000000 to more reasonable number.
We can also not set unsent-buffer-size at all.

@Only1Loatheb
Copy link

synchronized { should be kept in in WriterDataBuffer

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

@Only1Loatheb Only1Loatheb Awaiting requested review from Only1Loatheb

Assignees

@JanekFiranek JanekFiranek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JanekFiranek @Only1Loatheb