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

Keep track of timestamp_field mapping as part of a data stream #58400

Merged
merged 3 commits into from
Jun 22, 2020

Conversation

martijnvg
Copy link
Member

Backporting #58096 to 7.x branch.
Relates to #53100

  • use mapping source directly instead of using mapper service to extract the relevant mapping details
  • moved assertion to TimestampField class and added helper method for tests
  • Improved logic that inserts timestamp field mapping into an mapping.
    If the timestamp field path consisted out of object fields and
    if the final mapping did not contain the parent field then an error
    occurred, because the prior logic assumed that the object field existed.

…ic#58096)

Backporting elastic#58096 to 7.x branch.
Relates to elastic#53100

* use mapping source direcly instead of using mapper service to extract the relevant mapping details
* moved assertion to TimestampField class and added helper method for tests
* Improved logic that inserts timestamp field mapping into an mapping.
If the timestamp field path consisted out of object fields and
if the final mapping did not contain the parent field then an error
occurred, because the prior logic assumed that the object field existed.
@martijnvg martijnvg added backport :Data Management/Data streams Data streams and their lifecycles v7.9.0 labels Jun 22, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Data streams)

@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Data Management/Data streams Data streams and their lifecycles Team:Data Management Meta label for data/management team v7.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants