You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This means that it doesn't recognize the comma, even though it is expecting one. The parsing is stopped when reaching the comma.
Looking at the parse tree, the comma is parsed as <COMMAND_TOP_MODE_COMMA>, and the grammar rule is waiting for a "COMMA". The parser is probably in the wrong mode when it's parsing the comma.
A workaround is to not use the comma in between, as it is optional.
Expected behavior
Should parse with the comma in between fields as well.
How to reproduce
Test the query above.
Screenshots
Software version
6.1.0
Desktop (please complete the following information if relevant):
OS:
Browser:
Version:
Additional context
There were two test payloads with multiple fields, one of which (top2.txt) has been changed to not use a comma in between the fields to see if that works.
topSyntaxParseTest in TopSyntaxTests.java has been disabled because of this issue in PR #52.
The text was updated successfully, but these errors were encountered:
Describe the bug
There is a test case for a query:
| top 1 showperc=F countfield=zzz ddd, ddd
Which results in the following error:
This means that it doesn't recognize the comma, even though it is expecting one. The parsing is stopped when reaching the comma.
Looking at the parse tree, the comma is parsed as <COMMAND_TOP_MODE_COMMA>, and the grammar rule is waiting for a "COMMA". The parser is probably in the wrong mode when it's parsing the comma.
A workaround is to not use the comma in between, as it is optional.
Expected behavior
Should parse with the comma in between fields as well.
How to reproduce
Test the query above.
Screenshots
Software version
6.1.0
Desktop (please complete the following information if relevant):
Additional context
There were two test payloads with multiple fields, one of which (top2.txt) has been changed to not use a comma in between the fields to see if that works.
topSyntaxParseTest in TopSyntaxTests.java has been disabled because of this issue in PR #52.
The text was updated successfully, but these errors were encountered: