Skip to content
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

[7.x][docs] Backport: Add abbreviated titles for a cleaner nav (#16537) #16743

Merged
merged 1 commit into from
Mar 3, 2020

Conversation

dedemorton
Copy link
Contributor

Backports #16537 to 7.x branch.

@dedemorton dedemorton merged commit 9ba50e2 into elastic:7.x Mar 3, 2020
@dedemorton dedemorton deleted the backport_16537_7.x branch March 3, 2020 02:30
@chipzzz
Copy link

chipzzz commented Apr 20, 2020

@dedemorton , for add_fields

with the following config:

processors:
- add_fields:
    fields:
      test.newfield: dupa

you get fields.test.newfield as the field name. This is confusing as fields is part of the field name, where from the docs it looks like it's supposed to be used as a parameter.

This was noticed un functionbeat 7.x config

@chipzzz
Copy link

chipzzz commented Apr 20, 2020

@dedemorton , actually it's because of this

target (Optional) Sub-dictionary to put all fields into. Defaults to fields. I guess I must add target if I want my own top level field

@dedemorton
Copy link
Contributor Author

Yes, that's right. It looks like the published docs are correct. If you have additional feedback, please open a new GitHub issue rather than commenting on merged pull requests.

If you have questions, please post them to the discuss forum: https://discuss.elastic.co/c/beats. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants