Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[elasticsearch] remove unused default value #337

Merged
merged 1 commit into from
Oct 18, 2019
Merged

[elasticsearch] remove unused default value #337

merged 1 commit into from
Oct 18, 2019

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Oct 18, 2019

Related to #333 (comment)

@jmlrt jmlrt added the enhancement New feature or request label Oct 18, 2019
@jmlrt jmlrt requested a review from Crazybus October 18, 2019 10:13
@jmlrt jmlrt self-assigned this Oct 18, 2019
@jmlrt jmlrt mentioned this pull request Oct 18, 2019
Merged
7 tasks
Crazybus
Crazybus approved these changes Oct 18, 2019
View reviewed changes
Copy link
Contributor

@Crazybus Crazybus left a comment

Choose a reason for hiding this comment

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

LGTM!

@jmlrt
Copy link
Member Author

jmlrt commented Oct 18, 2019

jenkins test this please

@jmlrt jmlrt merged commit 88b12d3 into elastic:master Oct 18, 2019
@jmlrt jmlrt deleted the es-unused-default-value branch October 18, 2019 16:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Crazybus Crazybus Crazybus approved these changes

Assignees

@jmlrt jmlrt

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmlrt @Crazybus