teslamotors / fleet-telemetry

Apache License 2.0

625 stars 66 forks source link

Do not set opaque object if reliable ack workers are not set. This fixes a potential memory leak issue #153

Closed agbpatro closed 2 months ago

agbpatro commented 2 months ago

Description

Kafka allows you to send opaque entry which are used for post processing flows. we should only set that field if reliable ack workers are configured

Type of change

Please select all options that apply to this change:

[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation update

Checklist:

Confirm you have completed the following steps:

[X] My code follows the style of this project.
[X] I have performed a self-review of my code.
[ ] I have made corresponding updates to the documentation.
[ ] I have added/updated unit tests to cover my changes.
[ ] I have added/updated integration tests to cover my changes.