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

Indices.ExistsAsync throw exception due to ICU field #8181

Closed
feherbbj opened this issue May 6, 2024 · 6 comments
Closed

Indices.ExistsAsync throw exception due to ICU field #8181

feherbbj opened this issue May 6, 2024 · 6 comments
Labels
8.x Relates to 8.x client version Area: Generator Category: Bug

Comments

@feherbbj
Copy link

feherbbj commented May 6, 2024

Elastic.Clients.Elasticsearch version: 8.12.

Elasticsearch version: 8.12.2

.NET runtime version: 8.0

Description of the problem including expected versus actual behavior:
I have an ES instance with ICU plugin installed, when i call Indices.ExistsAsync(...) it throws exception because it cannot deserialize a property with field icu_collation_keyword to 'Mapping.IProperty'

Expected behavior
This used to work in 7.x

@feherbbj feherbbj added 8.x Relates to 8.x client version Category: Bug labels May 6, 2024
@flobernd
Copy link
Member

flobernd commented May 6, 2024

Hi @feherbbj,

could you please send me the response JSON payload so that I can reproduce the issue?

@feherbbj
Copy link
Author

feherbbj commented May 6, 2024

index is like:

{
   ...,
   "mappings": {
      "properties": {
         "itemname": {
            "type": "text",
            "fields": {
               ...
               "icu": { "type": "icu_collation_keyword" }
            }
         }
      }
   }
}

The repsonse is for await client.Indices.GetAsync(Indices.All):
Elastic.Transport.UnexpectedTransportException : Encountered an unsupported variant tag 'icu_collation_keyword' on 'Mapping.IProperty', which could not be deserialized.
----> System.Text.Json.JsonException : Encountered an unsupported variant tag 'icu_collation_keyword' on 'Mapping.IProperty', which could not be deserialized.
at Elastic.Transport.DefaultHttpTransport1.ThrowUnexpectedTransportException[TResponse](Exception killerException, List1 seenExceptions, RequestData requestData, TResponse response, RequestPipeline pipeline)
at Elastic.Transport.DefaultHttpTransport`1.RequestCoreAsync[TResponse](Boolean isAsync, HttpMethod method, String path, PostData data, RequestParameters requestParameters, OpenTelemetryData openTelemetryData, CancellationToken cancellationToken)

@flobernd
Copy link
Member

flobernd commented May 6, 2024

Thank you! I was able to identify the problem. Some generator modifications are required here.

@flobernd
Copy link
Member

flobernd commented May 8, 2024

@feherbbj Could you please double check this as well in 8.13.x when you are ready to migrate? This should already be fixed as well.

Copy link
Contributor

This issue is stale because it has been open 5 days with no activity. Remove stale label or comment or this will be closed in 2 days.

@feherbbj
Copy link
Author

i can confirm, this issue is fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.x Relates to 8.x client version Area: Generator Category: Bug
Projects
None yet
Development

No branches or pull requests

2 participants