-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Make the url field of heartbeat aggregatable. #6263
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
Some people may be relying on the fact that the URL is analyzed for their searching. I suggest we make this a multi-field so that there are beats/auditbeat/_meta/fields.common.yml Lines 23 to 31 in 4cbb1de
|
Sure. Try 2. |
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. Can you please add an entry to the CHANGELOG.asciidoc file.
Should be done. (there was some strange character on line 2391 in the asciidoc file, Cache: optional...) |
@perjahn Can you run |
Sorry, having some problems with my environment. |
Let us know if we can help. |
jenkins test it |
@perjahn This is close to being merged, if you want, we can either send you a PR to your branch after running |
Yes, please take over the commits. |
This takes over the PR elastic#6263
This takes over the PR #6263
No description provided.