Do not discard whole message if one of the LTSV name/value is missing

awslabs / flowgger

A fast data collector in Rust

BSD 2-Clause "Simplified" License

834 stars 55 forks source link

Do not discard whole message if one of the LTSV name/value is missing #30

Closed pdepaepe closed 5 years ago

pdepaepe commented 5 years ago

Hello,

Actually, if a name or a value is missing in a LTSV record, the whole message is droped. If it respect the LTSV specification, i think we should save the message by removing only the pair. This patch also print more helpful messages in console when such situation is encountered.

Thanks for the review, merge & release time.

Signed-off-by: Pierre De Paepe pierre.de-paepe@corp.ovh.com

crisidev commented 5 years ago

We merged this PR during the transition to awslabs. Closing.