Skip to content
This repository has been archived by the owner on Aug 24, 2022. It is now read-only.

Support ES 0.90.0 - mostly #8

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

Conversation

vizanto
Copy link

@vizanto vizanto commented Jun 6, 2013

Hey!

After these changes I got lein test passing with ElasticSearch 0.90.1, but had to comment nodes-stats because it gave me this exception:

Method signature problem with method indices from class class org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest {
  :class org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest,
  :method "indices",
  :types #{boolean org.elasticsearch.action.admin.indices.stats.CommonStatsFlags},
  :count 2,
  :all-methods [#<Method public org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest.indices(boolean)> #<Method public org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest org.elasticsearch.action.admin.cluster.node.stats.NodesStatsRequest.indices(org.elasticsearch.action.admin.indices.stats.CommonStatsFlags)>]}

Any possibility to prefer an overload with gavagai? Couldn't find any options or docs about this.

For your convenience, the overloads:
https://github.com/elasticsearch/elasticsearch/blob/master/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsRequest.java#L87

BlueXiphoid and others added 6 commits December 17, 2012 10:45
What's up doc!
Fixes:
- 0.90 isField, getField, setField API changes.
- convert-fields only returning first value from
  arrays because:
  org.elasticsearch.index.get.GetField.getValue()
  only ever returns the first value…

Removes:
- make-compatible-decode-smile
- nodes-stats: I don't know how to choose overload
@vizanto
Copy link
Author

vizanto commented Sep 23, 2013

Did you get a chance to look into this?
Updated to 0.90.5 yesterday.

aroemers added 3 commits December 13, 2013 13:38
Used this to add support for the `docAsUpsert` method of UpdateRequest.
The tests have been extended accordingly (plus fixed some older ones).
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.

2 participants