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.

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

lwc: remove some monitorFlow steps #1489

Merged
merged 1 commit into from
Nov 12, 2022

Conversation

brharrington
Copy link
Contributor

When processing data with a high input volume the
ConnectionSources, InputLines, and LwcDatapoints monitors would have a high throughput and the same number of elements flowing through. This change removes two of them so there is now just InputLines. This helps reduce overhead in the hot path.

When processing data with a high input volume the
`ConnectionSources`, `InputLines`, and `LwcDatapoints`
monitors would have a high throughput and the same
number of elements flowing through. This change removes
two of them so there is now just `InputLines`. This
helps reduce overhead in the hot path.
@brharrington brharrington added this to the 1.7.0 milestone Nov 12, 2022
@brharrington brharrington merged commit 358aef3 into Netflix:main Nov 12, 2022
@brharrington brharrington deleted the rm-monitors branch November 12, 2022 20:37
manolama pushed a commit to manolama/atlas that referenced this pull request May 22, 2024
When processing data with a high input volume the
`ConnectionSources`, `InputLines`, and `LwcDatapoints`
monitors would have a high throughput and the same
number of elements flowing through. This change removes
two of them so there is now just `InputLines`. This
helps reduce overhead in the hot path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant