-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Support the new Influx Line Protocol Parser #9474
Copy link
Copy link
Closed
Labels
feature requestRequests for new plugin and for new features to existing pluginsRequests for new plugin and for new features to existing pluginsperformanceproblems with decreased performance or enhancements that improve performanceproblems with decreased performance or enhancements that improve performanceplugin/parser1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins
Description
Feature Request
@rogpeppe has written a new Influx Line Protocol Parser that is more semantically correct and significantly faster, supporting zero-allocation parsing. We should upgrade to it.
Updated parser can be found here https://pkg.go.dev/github.com/influxdata/line-protocol/v2/influxdata
Proposal:
upgrade to the faster parser
Current behavior:
Current parser does unnecessary memory allocations, causing a performance hit compared to the new parser.

Desired behavior:
Upgrade to the new parser
Use case:
memory allocations are expensive and slow. No need to do unnecessary ones.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
feature requestRequests for new plugin and for new features to existing pluginsRequests for new plugin and for new features to existing pluginsperformanceproblems with decreased performance or enhancements that improve performanceproblems with decreased performance or enhancements that improve performanceplugin/parser1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins1. Request for new parser plugins 2. Issues/PRs that are related to parser plugins