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

Pre-release 8.8.0 cannot work with string fields longer than 5000000 characters #96036

Closed
droberts195 opened this issue May 11, 2023 · 1 comment · Fixed by #96031
Closed
Labels
:Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team v8.8.0

Comments

@droberts195
Copy link
Contributor

droberts195 commented May 11, 2023

#95641 upgraded Jackson to version 2.15.0. One of the fixes in this is FasterXML/jackson-core#863. This new limit of 5000000 characters for a string field then applies to strings in Elasticsearch documents.

To reproduce:

  1. ./gradlew run
  2. Download big_a.zip
  3. unzip big_a.zip
  4. curl -u elastic:password http://localhost:9200/my-index/_doc/1 -X PUT -d @big_a.json -H 'Content-Type: application/json'

The response is:

{"error":{"root_cause":[{"type":"document_parsing_exception","reason":"[1:20] failed to parse: String length (5043368) exceeds the maximum length (5000000)"}],"type":"document_parsing_exception","reason":"[1:20] failed to parse: String length (5043368) exceeds the maximum length (5000000)","caused_by":{"type":"stream_constraints_exception","reason":"String length (5043368) exceeds the maximum length (5000000)"}},"status":400}

I think this is a blocker for release of 8.8.0 because people will have indexed docs in older versions that have strings longer than 5000000 characters, and they won't be able to work with them in 8.8.0 unless we do something about this problem.

@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label May 11, 2023
@droberts195 droberts195 added :Core/Infra/Core Core issues without another label v8.8.0 and removed needs:triage Requires assignment of a team area label labels May 11, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label May 11, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team v8.8.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants