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

[pkg/ottl] Enhance ParseXML function with intuitive parse format and add MarshalXML function to convert parsed output back to XML #35076

Open
shazlehu opened this issue Sep 9, 2024 · 3 comments
Labels
enhancement New feature or request needs triage New item requiring triage pkg/ottl

Comments

@shazlehu
Copy link

shazlehu commented Sep 9, 2024

Component(s)

pkg/ottl

Is your feature request related to a problem? Please describe.

The current implementation of the ParseXML produces output that is difficult for users to manipulate, particularly when the XML is complicated. The current design loses the sense of key:value pairs which is helpful for understanding the data, and, more importantly, parses the XML into arrays, which are difficult to manipulate in OTTL, and don't provide a stable path that can be used to access the data.

Describe the solution you'd like

PR 35210 has a proposed new parsing version that maintains backward compatibility with the current implementation. (Updated to point to PR off observIQ repo)

Describe alternatives you've considered

No response

Additional context

No response

Copy link
Contributor

github-actions bot commented Sep 9, 2024

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@djaglowski
Copy link
Member

For context, discussion about the current design can be found on #31133.

@djaglowski
Copy link
Member

I'm going to make a few comments on #35210 but I think it's going to be worth bringing the discussion back here to focus on proposed changes to the format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs triage New item requiring triage pkg/ottl
Projects
None yet
Development

No branches or pull requests

2 participants