Josh,
Thank you for this PR. Would you be able to open an issue on our JIRA as well. Can you
explain in a bit more detail how this patch helps?
Thank you, again.
Best,
Tim
-----Original Message-----
From: joshbooks [mailto:git@git.apache.org]
Sent: Wednesday, January 18, 2017 4:15 PM
To: dev@tika.apache.org
Subject: [GitHub] tika pull request #148: be more parsimonious wrapping streams
GitHub user joshbooks opened a pull request:
https://github.com/apache/tika/pull/148
be more parsimonious wrapping streams
it looks like a bunch of streams were getting wrapped in BufferedInputStreams just to
make extra double sure that mark was supported, but this is not as harmless as it might otherwise
seem when you run into big nested package files
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/joshbooks/tika master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tika/pull/148.patch
To close this pull request, make a commit to your master/trunk branch with (at least) the
following in the commit message:
This closes #148
----
commit 896c46a0c652de436da0e4f25bfa53a7d83ae02f
Author: Joshua Hight <josh.hight@thinair.com>
Date: 2017-01-18T21:10:03Z
be more parsimonious wrapping streams
it looks like a bunch of streams were getting wrapped in
BufferedInputStream just to make extra double sure that mark was
supported, but this is not as harmless as it might otherwise seem when
you run into big nested package files
commit 9477d03e10149a8ec6b5d6889e2fd2317d2ed5f5
Author: Joshua Hight <josh.hight@thinair.com>
Date: 2017-01-18T21:13:05Z
Merge remote-tracking branch 'apache/master'
----
---
If your project is set up for it, you can reply to this email and have your reply appear on
GitHub as well. If your project does not have this feature enabled and wishes so, or if the
feature is enabled but not working, please contact infrastructure at infrastructure@apache.org
or file a JIRA ticket with INFRA.
---
|