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

Updated elasticsearch to 2.2.0 #257

Closed
wants to merge 1 commit into from
Closed

Updated elasticsearch to 2.2.0 #257

wants to merge 1 commit into from

Conversation

wombat
Copy link
Contributor

@wombat wombat commented Feb 17, 2016

No description provided.

@jnioche
Copy link
Contributor

jnioche commented Feb 17, 2016

Thanks Daniel. Don't the mapping need changing as well? I think TTL has been removed with 2.2.0
Moving to 2.x would probably require a change to the latest version of Kibana. Have you tried the existing dashboard definitions with it?

@wombat
Copy link
Contributor Author

wombat commented Feb 17, 2016

@jnioche Sorry, just removed the _id field mapping in my local application - added it now to the codebase. _ttl is deprecated since 2.0, but it generates no error or similar. I am also not aware of an alternative

@wombat
Copy link
Contributor Author

wombat commented Feb 25, 2016

@jnioche I updated to your latests commits and resolved a merge conflict that occurred recently

@jnioche jnioche added this to the 0.10 milestone Feb 26, 2016
@jnioche jnioche self-assigned this Feb 26, 2016
@jnioche
Copy link
Contributor

jnioche commented Feb 26, 2016

thanks @w0mbat. This is quite a big change for the ES module and I'd rather do that straight after releasing 0.9 so that it gets used as much as possible before releasing it. Does that make sense?

BTW Would a REST-based client (see #221) like Jest work on ES 1.x as well as 2.x?

@wombat
Copy link
Contributor Author

wombat commented Feb 26, 2016

@jnioche I can understand the decision to delay it to 0.10
I haven´t used Jest so far, but depending on the jest version, they are claiming to be compatible with ES 1.x and 2.x: https://github.com/searchbox-io/Jest

@jnioche
Copy link
Contributor

jnioche commented Mar 16, 2016

Note to self - breaking changes in 2.x

@w0mbat getting Exception "path.home is not configured" when using embedded mode i.e not specifying any hosts. I presume you are getting this as well?

@wombat
Copy link
Contributor Author

wombat commented Mar 16, 2016

@jnioche well, I haven´t seen this one so far. Does it only come with debug logging?

@jnioche
Copy link
Contributor

jnioche commented Mar 16, 2016

No, it is big bad exception which crashes the topo and it comes both on the UI and the logs with level ERROR

2016-03-16 16:26:16.033 b.s.d.executor [ERROR] 
java.lang.RuntimeException: java.lang.IllegalStateException: path.home is not configured
    at com.digitalpebble.storm.crawler.elasticsearch.persistence.StatusUpdaterBolt.prepare(StatusUpdaterBolt.java:190) ~[stormjar.jar:?]
    at backtype.storm.daemon.executor$fn__5694$fn__5707.invoke(executor.clj:757) ~[storm-core-0.10.0.jar:0.10.0]
    at backtype.storm.util$async_loop$fn__545.invoke(util.clj:477) [storm-core-0.10.0.jar:0.10.0]
    at clojure.lang.AFn.run(AFn.java:22) [clojure-1.6.0.jar:?]
    at java.lang.Thread.run(Thread.java:745) [?:1.7.0_80]
Caused by: java.lang.IllegalStateException: path.home is not configured
    at org.elasticsearch.env.Environment.<init>(Environment.java:101) ~[stormjar.jar:?]
    at org.elasticsearch.node.internal.InternalSettingsPreparer.prepareEnvironment(InternalSettingsPreparer.java:81) ~[stormjar.jar:?]
    at org.elasticsearch.node.Node.<init>(Node.java:128) ~[stormjar.jar:?]
    at org.elasticsearch.node.NodeBuilder.build(NodeBuilder.java:145) ~[stormjar.jar:?]
    at org.elasticsearch.node.NodeBuilder.node(NodeBuilder.java:152) ~[stormjar.jar:?]
    at com.digitalpebble.storm.crawler.elasticsearch.ElasticSearchConnection.getClient(ElasticSearchConnection.java:71) ~[stormjar.jar:?]
    at com.digitalpebble.storm.crawler.elasticsearch.ElasticSearchConnection.getConnection(ElasticSearchConnection.java:130) ~[stormjar.jar:?]
    at com.digitalpebble.storm.crawler.elasticsearch.persistence.StatusUpdaterBolt.prepare(StatusUpdaterBolt.java:186) ~[stormjar.jar:?]
    ... 4 more

jnioche added a commit that referenced this pull request Apr 1, 2016
@wombat wombat closed this Apr 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants