Skip to content

Conversation

@pjfanning
Copy link
Member

@pjfanning pjfanning commented Aug 30, 2022

https://issues.apache.org/jira/browse/TIKA-3843

Before opening the pull request, please verify that

  • commits are squashed into a single one (or few commits for larger changes)
  • Tika is successfully built and unit tests pass by running mvn clean test
  • there should be no conflicts when merging the pull request branch into the recent main branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled main branch
  • if you add new module that downstream users will depend upon add it to relevant group in tika-bom/pom.xml.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Tika in general, please sign up for the Tika mailing list. Thanks!

@pjfanning pjfanning marked this pull request as draft August 30, 2022 21:24
@pjfanning pjfanning changed the title use commons-io byte streams WIP: use commons-io byte streams (not ready to merge) Aug 30, 2022
@pjfanning pjfanning changed the title WIP: use commons-io byte streams (not ready to merge) [TIKA-3843] use commons-io byte streams (not ready to merge) Aug 30, 2022
@pjfanning pjfanning changed the title [TIKA-3843] use commons-io byte streams (not ready to merge) [TIKA-3843] use commons-io byte streams Aug 30, 2022
@pjfanning pjfanning marked this pull request as ready for review August 30, 2022 23:21
@pjfanning pjfanning force-pushed the unsynch-byte-array-stream branch 2 times, most recently from 9880620 to 814a547 Compare August 31, 2022 12:47
imports

Update OpenDocumentDetector.java

imports

revert dbf change

tika-core

avoid string-byte conversions

Update TikaPipesHttpClient.java

more changes

undo some changes
@pjfanning pjfanning force-pushed the unsynch-byte-array-stream branch from 814a547 to e645478 Compare August 31, 2022 18:25
@tballison tballison merged commit 9cbe0c9 into apache:main Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants