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

Cherry-pick #17614 to 7.7: Add static mapping for dynamodb metricset #17706

Merged
merged 1 commit into from
Apr 14, 2020

Conversation

ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Apr 14, 2020

Cherry-pick of PR #17614 to 7.7 branch. Original message:

What does this PR do?

This PR adds static mappings for aws dynamodb metricset.

Why is it important?

To avoid having breaking non TSVB visualisations when there no data in ES to create the respective fields with the dynamic mapping.

* Add static mapping for dynamodb metricset

Signed-off-by: chrismark <chrismarkou92@gmail.com>

* Fix field types

Signed-off-by: chrismark <chrismarkou92@gmail.com>
(cherry picked from commit e023845)
@ChrsMark
Copy link
Member Author

@exekias, any objections for this backport to 7.7 🙂 ?

@exekias
Copy link
Contributor

exekias commented Apr 14, 2020

SGTM!

@ChrsMark ChrsMark merged commit d120dd2 into elastic:7.7 Apr 14, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
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