Skip to content

chore(deps): Bump github.com/antchfx/jsonquery from v1.3.3 to v1.3.4-0.20240315124402-5db0be839244 #15004

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

chore(deps): Bump github.com/antchfx/jsonquery from v1.3.3 to v1.3.4-0.20240315124402-5db0be839244 #15004

wants to merge 1 commit into from

Conversation

srebhan
Copy link
Member

@srebhan srebhan commented Mar 15, 2024

Summary

Bump json-query library to handle nested arrays correctly.

Checklist

  • No AI generated code was used in this PR

Related issues

resolves #14976

@telegraf-tiger telegraf-tiger bot added the chore label Mar 15, 2024
@srebhan srebhan mentioned this pull request Mar 15, 2024
Closed
@powersj
Copy link
Contributor

powersj commented Mar 20, 2024

To make sure I am following the impact of this change, assuming you have the following data:

"phases": [
        {
          "load": 34,
          "voltage": 231
        },
        {
          "load": 35,
          "voltage": 231
        },
        {
          "load": 36,
          "voltage": 231
        }
      ],

Currently this produces:

phases_0_load
phases_0_voltage

And with this PR:

phases_phases0_load
phases_phases0_voltage

The reason for the change is for data like:

"values": [
	[
		1,
		2,
		3
	]
]

Currently, this does not work at all. With the PR, what does this produce?

@powersj powersj closed this Mar 20, 2024
@srebhan srebhan mentioned this pull request Mar 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@srebhan srebhan

Labels
chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Kapacitor Alert Parsing
2 participants
@srebhan @powersj