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
elasticsearchPlugins: add ingest-attachment #73150
elasticsearchPlugins: add ingest-attachment #73150
Conversation
so, i realized that this fails with edit: they're also broken for elasticsearch 7 due to the same version mismatch issue. does nobody use elasticsearch plugins? |
I'm not sure who uses them, I was using them when I was still using es on nixos. Can you please update them, looks like someone forgot to update plugins when updating es? Can you also please add comment to elasticsearch that plugins needs to be updated when updating package? |
no problem. i'll also remove the 5.x versions since elasticsearch 5.x has been removed. also, i noticed that some of the plugins use |
i've pushed some style fixups and stuff as well. if a reviewer wants those split out of this pr i'd be happy to do so. |
looks good to me lgtm |
Btw @ashkitten have you tested this with nixos module with latest elasticsearch? Does it work well? |
i've tested it with default elasticsearch and it works for my use case, at least. |
Motivation for this change
needed for using elasticsearch for full text search in nextcloud (https://github.com/nextcloud/fulltextsearch_elasticsearch/wiki). tested it briefly to make sure it works in my use-case.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @offlinehacker