-
Notifications
You must be signed in to change notification settings - Fork 102
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
Events #436
Events #436
Conversation
Most of work is done already, i can think only about some refactoring. Let's treat it as RFC :) |
|
||
@Override | ||
public void handle(ChannelHandlerContext ctx, HttpRequest request) { | ||
final String tenantId = request.getHeader("tenantId"); |
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.
NIT: Can we extract these constants in Utils instead of hard coding them here?
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.
Sure, you've just helped me understand where they would go :)
Just not to decrease total coverage
This work is being continued in PR #477 |
This PR adds events (annotations). Events interface mimics
graphite-web
API.POST
request at/events/
with serialized event as request body.GET
request at/events/get_data
. Returns array of events. Acceptsfrom
,until
,tags
query parameters.Store events in
ElasticSearch
withtenant-id
as routing parameter.