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

Fix missing dependencies for Bulk and Scroll #3

Closed
wants to merge 1 commit into from
Closed

Fix missing dependencies for Bulk and Scroll #3

wants to merge 1 commit into from

Conversation

@choroba
Copy link

@choroba choroba commented Jan 14, 2018

Search::Elasticsearch::Bulk and Scroll don't exist anymore. They were
replaced by Search::Elasticsearch::Client::6_0::Bulk and Scroll,
respectively.

Search::Elasticsearch::Bulk and Scroll don't exist anymore. They were
replaced by Search::Elasticsearch::Client::6_0::Bulk and Scroll,
respectively.
@mickeyn
Copy link

@mickeyn mickeyn commented Jan 15, 2018

Thanks for the change suggestions, but -

This distribution is semi-deprecated... we only use it in MetaCPAN but are in the process of removing it. We also don't use ES 6 or any of the > 2.x versions because of (no) backwards-compatibility.

At this point, merging such change is too risky for us.

@mickeyn mickeyn closed this Apr 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants