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.

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

Root objects are not considered valid paths in Remap #6266

Closed
binarylogic opened this issue Jan 27, 2021 · 1 comment · Fixed by #6353
Closed

Root objects are not considered valid paths in Remap #6266

binarylogic opened this issue Jan 27, 2021 · 1 comment · Fixed by #6353
Assignees
@JeanMertz
Labels
domain: vrl Anything related to the Vector Remap Language type: bug A code related bug.
Milestone
2021-02-01 D-Fuel

Comments

@binarylogic
Copy link
Contributor

You cannot pass the root object to functions that support working with objects:

Configuration error. error=Transform "compact": remap error: parser error:  --> 1:16
  |
1 |   . = compact(.)␊
  |                ^---
  |
  = expected path_segment
@binarylogic binarylogic added type: bug A code related bug. domain: vrl Anything related to the Vector Remap Language labels Jan 27, 2021
@binarylogic binarylogic added this to the 2021-01-18 Tabula E-Rasa milestone Jan 27, 2021
@JeanMertz
Copy link
Contributor

Curious. This seems to be a regression...

I added a test in the new parser, and this is working as expected, so should be solved as soon as that PR lands.

@binarylogic binarylogic changed the title Root objects are not considered valid paths in Remapo Root objects are not considered valid paths in Remap Feb 3, 2021
@JeanMertz JeanMertz mentioned this issue Feb 4, 2021
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JeanMertz JeanMertz

Labels
domain: vrl Anything related to the Vector Remap Language type: bug A code related bug.
Projects
None yet
Milestone
2021-02-01 D-Fuel
Development

Successfully merging a pull request may close this issue.

chore(remap): re-implement VRL parser/compiler vectordotdev/vector
3 participants
@jamtur01 @JeanMertz @binarylogic