Skip to content

AttachmentProcessorTests.testIndexedChars fails when running on jdk 11ea #31305

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
alpar-t opened this issue Jun 13, 2018 · 6 comments
Closed
Labels
blocker :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP :Delivery/Build Build or test infrastructure jdk11 Team:Delivery Meta label for Delivery team

Comments

@alpar-t
Copy link
Contributor

alpar-t commented Jun 13, 2018

 Throwable #1: ElasticsearchParseException[Error parsing document in field [source_field]]; nested: TikaException[Unexpected RuntimeException from org.apache.tika.parser.txt.TXTParser@5bdd2e62]; nested: IllegalStateException[Can't overwrite cause with org.apache.tika.sax.WriteOutContentHandler$WriteLimitReachedException: Your document contained more than 19 characters, and so your requested limit has been reached. To receive the full text of the document, increase your limit. (Text up to the limit is however available).]; nested: TaggedSAXException[Your document contained more than 19 characters, and so your requested limit has been reached. To receive the full text of the document, increase your limit. (Text up to the limit is however available).]; nested: WriteLimitReachedException[Your document contained more than 19 characters, and so your requested limit has been reached. To receive the full text of the document, increase your limit. (Text up to the limit is however available).];
   >     at __randomizedtesting.SeedInfo.seed([F36010291A3A195A:F8D959C10745D812]:0)
   >     at org.elasticsearch.ingest.attachment.AttachmentProcessor.execute(AttachmentProcessor.java:106)
   >     at org.elasticsearch.ingest.attachment.AttachmentProcessorTests.parseDocument(AttachmentProcessorTests.java:285)
   >     at org.elasticsearch.ingest.attachment.AttachmentProcessorTests.parseDocument(AttachmentProcessorTests.java:275)
   >     at org.elasticsearch.ingest.attachment.AttachmentProcessorTests.testIndexedChars(AttachmentProcessorTests.java:296)

Output of java -version:

openjdk version "11-ea" 2018-09-25
OpenJDK Runtime Environment 18.9 (build 11-ea+17)
OpenJDK 64-Bit Server VM 18.9 (build 11-ea+17, mixed mode)

Reproduce with:

 ./gradlew :plugins:ingest-attachment:test  -Dtests.class=org.elasticsearch.ingest.attachment.AttachmentProcessorTests -Dtests.method="testIndexedChars"

Note that running with JDK 11 requires some changes before one can get here. Those can be found here:
https://github.com/atorok/elasticsearch/tree/upgrade/jdk_11

@alpar-t alpar-t added :Delivery/Build Build or test infrastructure :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP labels Jun 13, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@tballison
Copy link

Thank you for raising this. @dadoonet, thank you for the ping. I can reproduce it in tika-core with Java 11-ea. I've opened: TIKA-2668.

@tballison
Copy link

This is now fixed and will be available in Tika 1.19.

@dadoonet
Copy link
Member

Awesome! Thanks @tballison!

@tballison
Copy link

As mentioned on other tickets, Tika 1.19 is now available. Let us know what you find.

jkakavas added a commit that referenced this issue Oct 12, 2018
Upgrades Tika to 1.19.1 and relevant transitive dependencies

Resolves: #31456, #31305
jkakavas added a commit that referenced this issue Oct 12, 2018
Upgrades Tika to 1.19.1 and relevant transitive dependencies

Resolves: #31456, #31305
jkakavas added a commit that referenced this issue Oct 12, 2018
Upgrades Tika to 1.19.1 and relevant transitive dependencies

Resolves: #31456, #31305
@jkakavas
Copy link
Member

Resolved by #33896

kcm pushed a commit that referenced this issue Oct 30, 2018
Upgrades Tika to 1.19.1 and relevant transitive dependencies

Resolves: #31456, #31305
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP :Delivery/Build Build or test infrastructure jdk11 Team:Delivery Meta label for Delivery team
Projects
None yet
Development

No branches or pull requests

6 participants