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

Influx Tag mapper (#1513) #2184

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LukasVyhlidka
Copy link

Pull request for issue #1513

Copy link
Contributor

@PyvesB PyvesB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I can tell, this would do the job, though I believe there's a potential problem with escaping of field values. If t.getValue() itself contains quotes, those would need to be escaped when using the line protocol. Indeed, according to the InfluxDB documentation:

For string field values use a backslash character \ to escape double quotes "

@tommcdo
Copy link

tommcdo commented Feb 1, 2021

Influx also supports fields of other types, so I wonder if it some specialized serialization would be useful here.

In my use case, I need to write multiple fields with numerical values.

@leonardehrenfried
Copy link

Hello all, I was also hit with the "high series cardinality" problem in influxdb and this seems like the correct solution.

If I resolved the merge conflicts, fixed the escaping and resubmitted it, would there be a chance that it will be merged?

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.

4 participants