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

Support usage of custom builders without hints and mappers #13839

Merged
merged 2 commits into from
Oct 28, 2019

Conversation

vjsamuel
Copy link
Contributor

Currently the code expects for a mapping template or hints to be enabled. There are cases where custom builders can be used which is enough to make autodiscover work. This PR allows that. This used to work but for some reason it is broken now.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@vjsamuel vjsamuel force-pushed the support_custom_builders branch from 6dace9c to 841d37e Compare September 27, 2019 21:31
@urso
Copy link

urso commented Sep 27, 2019

The config construction is relatively complex. Maybe we should add a test, to prevent further regressions in the future.

@vjsamuel vjsamuel force-pushed the support_custom_builders branch from 841d37e to ce6d0c6 Compare October 25, 2019 05:42
@vjsamuel
Copy link
Contributor Author

@urso i have added the missing test case.

@vjsamuel vjsamuel force-pushed the support_custom_builders branch from ce6d0c6 to 1db1cd5 Compare October 25, 2019 05:44
@vjsamuel vjsamuel force-pushed the support_custom_builders branch from 1db1cd5 to 79c6e91 Compare October 25, 2019 20:48
@exekias
Copy link
Contributor

exekias commented Oct 25, 2019

ok to test

@exekias exekias merged commit 414dd14 into elastic:master Oct 28, 2019
jorgemarey pushed a commit to jorgemarey/beats that referenced this pull request Jun 8, 2020
…3839)

* Support usage of custom builders without hints and mappers
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.

4 participants