Skip to content

docvalue_fetcher for icu_collation_keyword fields fails totally #70276

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

Closed
nik9000 opened this issue Mar 10, 2021 · 3 comments
Closed

docvalue_fetcher for icu_collation_keyword fields fails totally #70276

nik9000 opened this issue Mar 10, 2021 · 3 comments
Assignees
Labels
>bug :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team

Comments

@nik9000
Copy link
Member

nik9000 commented Mar 10, 2021

Elasticsearch version (bin/elasticsearch --version): master, more?

curl -XDELETE -uelastic:password -HContent-Type:application/json localhost:9200/test
curl -XPUT -uelastic:password -HContent-Type:application/json localhost:9200/test -d'{
  "mappings": {
    "properties": {
      "k": {
        "type": "icu_collation_keyword"
      }
    }
  }
}'
curl -XPOST -uelastic:password -HContent-Type:application/json 'localhost:9200/test/_doc?pretty&refresh' -d'{
  "k": "the quick brown fox"
}'
curl -XPOST -uelastic:password -HContent-Type:application/json localhost:9200/test/_search?pretty -d'{
  "fields": [{"field": "k"}]
}'
# "fields" : {"k": ["the quick brown fox"]}  Great!

curl -XPOST -uelastic:password -HContent-Type:application/json localhost:9200/test/_search?pretty -d'{
  "docvalue_fields": [{"field": "k"}]
}'
# "fields" : {"k" : ["⠜ಁॊ⎢燰Ⴑᢌ噄Țᆖ\"瀑㠀\u0001"]}
@nik9000 nik9000 added >bug needs:triage Requires assignment of a team area label labels Mar 10, 2021
@nik9000
Copy link
Member Author

nik9000 commented Mar 10, 2021

Well, I guess it isn't fair to say they fail totally. I mean, nothing crashes. Its just wrong.

@nik9000 nik9000 added :Search/Search Search-related issues that do not fall into other categories and removed needs:triage Requires assignment of a team area label labels Mar 10, 2021
@elasticmachine elasticmachine added the Team:Search Meta label for search team label Mar 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search (Team:Search)

@jtibshirani
Copy link
Contributor

As noted in #53246, we decided not to invest in fixing these issues with docvalue_fields. The recommended way to load these values is through the new fields option instead. As part of our testing efforts around field loading (#70929) we could look into throwing clearer errors for field types that don't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team
Projects
None yet
Development

No branches or pull requests

4 participants