Skip to content

Adapting to ES 2.0.0 #21

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adapting to ES 2.0.0 #21

wants to merge 2 commits into from

Conversation

antonha
Copy link

@antonha antonha commented Nov 13, 2015

ES 2.0.0 is out, and I'd like to contribute to getting this plugin into that release.

I did some refactoring in the ComboAnalyzerClass, but most of the things are either changes required by Lucene 5.x or ES 2.x

@@ -76,6 +77,19 @@
</dependency>

<dependency>
<groupId>junit</groupId>
Copy link
Author

Choose a reason for hiding this comment

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

This was needed due to ES checking for 'jar hell' when booting, even in test. Junit brings in hamcrest-core, which conflicts with the usage of hamcrest-all in the project's tests.

@antonha
Copy link
Author

antonha commented Nov 13, 2015

Any feedback would be very welcome. Most likely, there is some edge case that I've missed.

@adrianocrestani
Copy link

+1

@imranazad
Copy link

I tried running the following command:

plugin.bat install "file:///elasticsearch-2.0.0/plugins/elasticsearch-analysis-combo-es20.zip"

but I'm still getting an error regarding 'plugin-descriptor.properties' even though I can see it exists.

@imranazad
Copy link

I have built the plugin and I'm now getting the following error:

ERROR: C:\elasticsearch-2.0.0\plugins\elasticsearch-analysis-combo-2.0.0-SNAPSHOT.zip\plugin-descriptor.properties

@antonha
Copy link
Author

antonha commented Dec 15, 2015

@imranazad The plugin installation from file works fine for me on osx, running ES 2.0.0:

❯ bin/plugin install "file:///Users/anton/Desktop/es-plugin-test/elasticsearch-analysis-combo-2.0.0-SNAPSHOT.zip"
-> Installing from file:/Users/anton/Desktop/es-plugin-test/elasticsearch-analysis-combo-2.0.0-SNAPSHOT.zip...
Plugins directory [/Users/anton/Desktop/es-plugin-test/elasticsearch-2.0.0/plugins] does not exist. Creating...
Trying file:/Users/anton/Desktop/es-plugin-test/elasticsearch-analysis-combo-2.0.0-SNAPSHOT.zip ...
Downloading .DONE
Verifying file:/Users/anton/Desktop/es-plugin-test/elasticsearch-analysis-combo-2.0.0-SNAPSHOT.zip checksums if available ...
NOTE: Unable to verify checksum for downloaded plugin (unable to find .sha1 or .md5 file to verify)
Installed analysis-combo into /Users/anton/Desktop/es-plugin-test/elasticsearch-2.0.0/plugins/analysis-combo

If I guess correctly from your commands, it seems like you have placed the zip-file in the plugin directory, and then try to run the install script on that file? I don't believe that's how the plugin script is intended to be used. Could you try placing the built .zip-file somewhere else, and running the plugin script pointing to that file instead?

This could also be due to permission issues, or due to the plugin-script not working correctly on windows (but that would be highly unlikely).

Until further proof, I'm going to assume that this is not an issue with this plugin specifically.

@antonha
Copy link
Author

antonha commented Dec 15, 2015

@ofavre Are you still a maintainer of this repo? If so, could you leave a comment regarding this PR? If not, could you point me in the right direction?

Any help would be greatly appreciated :)

@rog
Copy link

rog commented Dec 19, 2015

I got the same error as @imranazad:

ERROR: Could not find plugin descriptor 'plugin-descriptor.properties' in plugin zip

I did a wget "https://github.com/antonha/elasticsearch-analysis-combo/archive/master.zip"

and then sudo bin/plugin install "file:///home/user/master.zip"

@antonha
Copy link
Author

antonha commented Dec 21, 2015

@RoGr This plugin is what ES calls a "Java Plugin", which means (among other things) that the zip-file to be installed needs to be built to contain .jar files with .class files in it. For this plugin, this file is built using maven.

You were trying to install a .zip archive containing the source code. This will never work. To install the plugin, you need to build the file from source (until it makes in into the main branch). Also, you should not be using the master branch of my fork. The es20 branch contains the necessary changes for ES 2.0.

@rog
Copy link

rog commented Dec 21, 2015

Yeah, I realised that I will built the plugin from your es20 branch and give you feedback..

Hope @yakaz or somebody merge your work to the main repo.

Thank you.

@imranazad
Copy link

@antonha Sorry for the late response, it's been manic recently. I placed the plugin in a different location and it worked! Many thanks, I really needed this plugin desperately.

@felixbarny
Copy link

@yakaz are you going to merge this? This would be really important to us.

@felixbarny
Copy link

felixbarny commented Feb 15, 2016

Seems like ES 2.2 needs another adjustment

java.lang.IllegalAccessException: Class org.elasticsearch.common.inject.DefaultConstructionProxyFactory$1 can not access a member of class org.elasticsearch.index.analysis.ComboAnalyzerProvider with modifiers ""

@vicmosin
Copy link

So did you guys fix it? I am still receiving

ERROR: Could not find plugin descriptor 'plugin-descriptor.properties' in plugin zip

during installation..

P.S. I have ES v2.1.0

@felixbarny
Copy link

Have you found an alternative to this plugin which works with current Elasticsearch versions? Unfortunately, this project does not seem to be maintained anymore :( I'm stuck on older versions because of it.

@vicmosin
Copy link

@felixbarny Unfortunately no..

@paulhyo
Copy link

paulhyo commented Jun 29, 2016

Has everybody found an alternative to this plugin which works with current Elasticsearch versions????

@Kijewski
Copy link

No, I'm still using the old ES version because of that. IMO this plugin is essential and should be adopted into the core. :-/

@waizuwolf
Copy link

I tried to port this plugin to ES 2.2.0 , I was able to build it. I made the request to ElasticSearch and got following error :
java.lang.NoClassDefFoundError: Could not initialize class org.apache.lucene.util.ReaderCloneFactory
I have posted all the logs @ StackOverflow
Any idea , why this is happening. It's working fine on ES 2.1.2

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.

10 participants