Yu Long comments

Results 18 comments of


                                            Yu Long

feat(parser.prometheusremotewrite): Add dense metric version to better support histograms

I think I addressed all the comments. Feel free to reopen if I left out anything.

feat(parser.prometheusremotewrite): Add dense metric version to better support histograms

Some comments on the folder-based test cases: - input file format: Prom RW with native histogram needs to be in protobuf format and I don't find an easy and readable...

feat(parser.prometheusremotewrite): Add dense metric version to better support histograms

By using 1) `jsonpb` to parse json input files 2) `influx.Parser` to parse influxdb line format expected output files We are able to greatly simplify the test setup logic. Thanks...

feat(parser.prometheusremotewrite): Add dense metric version to better support histograms

I think the broken integration test (on `output.sql`) is unrelated to my changes and already fixed in master by https://github.com/influxdata/telegraf/commit/edb28dea0715ab20c6f32ccd954db8601ba82c7a should I rebase or do we don't care?

feat(serializer.prometheusremotewrite): output native histogram

For sure, thank you.

feat(serializer.prometheusremotewrite): output native histogram

Yup, thank you. I will take another look at this one later. Will tag you when this is ready to be looked at :)

feat(serializer.prometheusremotewrite): output native histogram

Thanks for checking in. I'd love to iterate on this, will try to get to it this month...

feat(serializer.prometheusremotewrite): output native histogram

Finally got the chance to iterate on it; @srebhan thank you so much for the thoughtful feedback! the only CI broken is the same as [this one](https://github.com/influxdata/telegraf/pull/16493#issuecomment-2699960110) which already get...