-
Notifications
You must be signed in to change notification settings - Fork 455
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
[query] Add endpoint that parses query to an AST #2002
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2002 +/- ##
=========================================
+ Coverage 63% 63.5% +0.4%
=========================================
Files 1123 1119 -4
Lines 106854 106426 -428
=========================================
+ Hits 67419 67635 +216
+ Misses 35061 34455 -606
+ Partials 4374 4336 -38
Continue to review full report at Codecov.
|
return nodeMap, nil | ||
} | ||
|
||
func parsingThreshold(r *http.Request) bool { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dead code?
@@ -53,6 +53,7 @@ const ( | |||
timeParam = "time" | |||
queryParam = "query" | |||
debugParam = "debug" | |||
thresholdParam = "threshold" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like this is dead code too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM other than minor comments
What this PR does / why we need it:
Adds an endpoint that parses and returns a JSON representation of the given query.
Special notes for your reviewer:
Does this PR introduce a user-facing and/or backwards incompatible change?:
Does this PR require updating code package or user-facing documentation?: