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

Rename dataset.* to datastream.* #618

Merged
merged 4 commits into from
Aug 4, 2020
Merged

Conversation

mtojek
Copy link
Contributor

@mtojek mtojek commented Aug 4, 2020

This PR adjusts the validation to check datastream fields instead of dataset.

@mtojek mtojek self-assigned this Aug 4, 2020
@mtojek mtojek requested a review from ruflin August 4, 2020 09:37
@elasticmachine
Copy link

elasticmachine commented Aug 4, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #618 updated]

  • Start Time: 2020-08-04T10:03:54.812+0000

  • Duration: 5 min 36 sec

Test stats 🧪

Test Results
Failed 0
Passed 80
Skipped 0
Total 80

type: constant_keyword
description: >
Dataset type.
- name: dataset.name
- name: datastream.name
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is datastream.dataset now.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

type: constant_keyword
description: >
Dataset type.
- name: dataset.name
- name: datastream.name
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See above.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@mtojek mtojek merged commit 737fb54 into elastic:master Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants