Skip to content
This repository was archived by the owner on Nov 18, 2022. It is now read-only.

Elasticsearch 7 rest client - #58

Open
purplesword wants to merge 24 commits into
dropwizard:masterfrom
purplesword:elasticsearch_7_rest_client
Open

Elasticsearch 7 rest client#58
purplesword wants to merge 24 commits into
dropwizard:masterfrom
purplesword:elasticsearch_7_rest_client

Conversation

@purplesword

Copy link
Copy Markdown

No description provided.

@joschi

joschi commented May 26, 2019

Copy link
Copy Markdown
Member

@purplesword Thanks for your contribution!

Could you please resolve the conflicts with the master branch and remove Gradle from the PR? We're not planning to change the build system from Maven to Gradle.

@coveralls

coveralls commented May 26, 2019

Copy link
Copy Markdown

Coverage Status

Coverage increased (+8.6%) to 87.923% when pulling 4ce4c2e on purplesword:elasticsearch_7_rest_client into e2088de on dropwizard:master.

Comment thread .gitignore Outdated
Comment thread pom.xml Outdated
Comment thread pom.xml Outdated

@JsonProperty
@NotNull
private Map<String, String> settings = Collections.emptyMap();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

added a bunch more.. Not sure if did it properly.. Tests will be added soon.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I added more tests now the coverage is higher and 100% on health checks.

Could you comment more about what do you think is necessary to be covered in configuration?

I added most of them listed in the documentation (but feel a bit hard to write test unless using testcontainer). Also I sort of feel it's a bit too complicate, maybe it's better to cover some simple common configs and then let sophisticated users do what they want with the ManagedClient(client,sniffer) constructor with their own client and sniffer..

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants