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

Setting index_analyzer incorrectly sets analyzer #2329

Closed
clintongormley opened this issue Oct 15, 2012 · 1 comment
Closed

Setting index_analyzer incorrectly sets analyzer #2329

clintongormley opened this issue Oct 15, 2012 · 1 comment

Comments

@clintongormley
Copy link

Setting just index_analyzer incorrectly sets analyzer:

curl -XPUT 'http://127.0.0.1:9200/test/?pretty=1'  -d '
{
   "mappings" : {
      "test" : {
         "properties" : {
            "foo" : {
               "index_analyzer" : "whitespace",
               "type" : "string"
            }
         }
      }
   }
}
'

curl -XGET 'http://127.0.0.1:9200/test/test/_mapping?pretty=1' 

# {
#    "test" : {
#       "properties" : {
#          "foo" : {
#             "type" : "string",
#             "analyzer" : "whitespace"
#          }
#       }
#    }
# }
@kimchy
Copy link
Member

kimchy commented Oct 16, 2012

The logic here was that if you set the index_analyzer then you would almost always want to set the search_analyzer as well, so we do it automatically. But then, it was before we had the analyzer setting, now, its confusing, agreed. Will fix it.

@kimchy kimchy closed this as completed in a963bc1 Oct 16, 2012
kimchy added a commit that referenced this issue Oct 16, 2012
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
martijnvg pushed a commit to martijnvg/elasticsearch that referenced this issue Jan 31, 2018
* Moves more classes over to ToXContentObject/Fragment

* Removes ToXContentToBytes

* Removes ToXContent from Enums

* review comment fix

* slight change to use XContantHelper
martijnvg pushed a commit that referenced this issue Feb 5, 2018
* Moves more classes over to ToXContentObject/Fragment

* Removes ToXContentToBytes

* Removes ToXContent from Enums

* review comment fix

* slight change to use XContantHelper
martijnvg added a commit that referenced this issue Feb 5, 2018
* es/6.x: (155 commits)
  Make persistent tasks work. Made persistent tasks executors pluggable.
  Removed ClientHelper dependency from PersistentTasksService.
  Added AllocatedPersistentTask#waitForPersistentTaskStatus(...) that delegates to PersistentTasksService#waitForPersistentTaskStatus(...)
  Add adding ability to associate an ID with tasks.
  Remove InternalClient and InternalSecurityClient (#3054)
  Make the persistent task status available to PersistentTasksExecutor.nodeOperation(...) method
  Refactor/to x content fragments2 (#2329)
  Make AllocatedPersistentTask members volatile (#2297)
  Moves more classes over to ToXContentObject/Fragment (#2283)
  Adapt to upstream changes made to AbstractStreamableXContentTestCase (#2117)
  Move tribe to a module (#2088)
  Persistent Tasks: remove unused isCurrentStatus method (#2076)
  Call initialising constructor of BaseTasksRequest (#1771)
  Always Accumulate Transport Exceptions (#1619)
  Pass down the provided timeout.
  Fix static / version based BWC tests (#1456)
  Don't call ClusterService.state() in a ClusterStateUpdateTask
  Separate publishing from applying cluster states
  Persistent tasks: require allocation id on task completion (#1107)
  Fixes compile errors in Eclipse due to generics
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants