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

Terms query is broken #80

Closed
clintongormley opened this issue Mar 22, 2010 · 2 comments
Closed

Terms query is broken #80

clintongormley opened this issue Mar 22, 2010 · 2 comments

Comments

@clintongormley
Copy link

Hiya

Commit 93e0253 breaks the terms query:

curl -XPUT 'http://127.0.0.2:9200/es_test_1/'
curl -XPUT 'http://127.0.0.2:9200/_all/type_1/_mapping?ignoreConflicts=true'  -d '
{
   "properties" : {
      "num" : {
         "store" : "yes",
         "type" : "integer"
      },
      "text" : {
         "store" : "yes",
         "type" : "string"
      }
   }
}
'


curl -XGET 'http://127.0.0.2:9200/_cluster/health?timeout=2s&waitForStatus=yellow' 


curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/1'  -d '
{
   "num" : 2,
   "text" : "foo"
}
'

curl -XGET 'http://127.0.0.2:9200/_terms?fields=num&toInclusive=true' 
# {"error":null}
@kimchy
Copy link
Member

kimchy commented Mar 22, 2010

Terms query is broken, closed by f9cac39.

@clintongormley
Copy link
Author

Fixed ++

dadoonet added a commit that referenced this issue Jun 5, 2015
Described in #10614, it's not possible with cloud discovery plugin to update dynamic settings anymore.

```sh
curl -XPUT localhost:9200/_cluster/settings -d '{
    "persistent" : {
        "discovery.zen.minimum_master_nodes" : 3
    },
    "transient" : {
        "discovery.zen.minimum_master_nodes" : 3
    }
}'
```

gives

```json
{"acknowledged":true,"persistent":{},"transient":{}}
```

Closes #80.
(cherry picked from commit 7749055)
(cherry picked from commit 82bea69ef0f24c5f60a68bd823f638890b984691)

Conflicts:
	src/main/java/org/elasticsearch/discovery/azure/AzureDiscovery.java
rmuir pushed a commit to rmuir/elasticsearch that referenced this issue Nov 8, 2015
Closes elastic#80.

(cherry picked from commit 930c8be)
rmuir pushed a commit to rmuir/elasticsearch that referenced this issue Nov 8, 2015
Related to elastic#80.

(cherry picked from commit 89d5460)
rmuir pushed a commit to rmuir/elasticsearch that referenced this issue Nov 8, 2015
rmuir pushed a commit to rmuir/elasticsearch that referenced this issue Nov 8, 2015
With issue elastic#80 we explicitly removed appache POI dependency provided by Tika and replaced with a more recent one.
Sadly we forgot to add this new dependency to the assembly so the final ZIP file does not contain POI related jars.

Closes elastic#82.

(cherry picked from commit 49793d5)
maabernethy pushed a commit to maabernethy/elasticsearch that referenced this issue Mar 20, 2017
Changed from issue 18109 to 21978
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this issue Jan 3, 2018
henningandersen pushed a commit to henningandersen/elasticsearch that referenced this issue Jun 4, 2020
Due to elastic/rally#852 we will implement a compatibility layer in the
current load generator that will also use the asyncio API and thus
requires custom runners to be registered differently (by specifying
`async_runner=True`). Rally's runner registry will also expose a new
attribute `async_runner` that is set to `True` if Rally requires runners
to be registered as described above.

With this commit we introduce a (temporary) compatibility layer for all
custom runners that allows older Rally versions to work with the classic
runners and newer Rally versions with the async runners.

Relates elastic/rally#852
Relates elastic/rally#916
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Oct 2, 2023
This issue was closed.
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

No branches or pull requests

2 participants