-
Notifications
You must be signed in to change notification settings - Fork 30
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
Support for analyzers #357
Support for analyzers #357
Conversation
…ort-for-analyzers
…ture from ArangoDB 3.4 test commands
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.
Solid contribution as always, thanks @tjoubert ! I left a few comments to clarify minor items, but happy to merge once discussed.
Hey @DiscoPYF, thanks a lot for your review. I have addressed all the issues. |
…ort-for-analyzers
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.
Comments addressed and fixed. LGTM.
Added support for the Analyzers endpoints.
GET /_api/analyzer
List all AnalyzersPOST /_api/analyzer
Create an Analyzer with the supplied definitionDELETE /_api/analyzer/{analyzer-name}
Remove an AnalyzerGET /_api/analyzer/{analyzer-name}
Return the Analyzer definition