[GitHub] tika pull request: TIKA-1369 Avoid ThreadLocal usage from Memory L...
Github user vilmospapp closed the pull request at: https://github.com/apache/tika/pull/17 --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] tika pull request: TIKA-1369 Avoid ThreadLocal usage from Memory L...
GitHub user vilmospapp opened a pull request: https://github.com/apache/tika/pull/17 TIKA-1369 Avoid ThreadLocal usage from Memory Leak Hi @chrismattmann , Based on our discussion from https://github.com/apache/tika/pull/15 I've added the ThreadLocal clean up part, so theoretically it won't suffer from the scenario that @grossws mentioned. Cheers, Vilmos You can merge this pull request into a Git repository by running: $ git pull https://github.com/vilmospapp/tika TIKA-1369-2 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tika/pull/17.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 #17 commit f95fad94619946ef1d4fe7cf407deab6317ad2fd Author: Vilmos Papp Date: 2014-10-06T12:10:14Z TIKA-1369 Avoid ThreadLocal usage from Memory Leak --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] tika pull request: TIKA-1369 Resolve thread safety issue in ImageM...
GitHub user vilmospapp opened a pull request: https://github.com/apache/tika/pull/15 TIKA-1369 Resolve thread safety issue in ImageMetadataExtractor Hi, This fix tries to resolve TIKA-1369 with handle thread safety by ThreadLocal and avoid other library dependencies. I have run the test cases, so it seems correct to me, though I haven't found any other occurrence of ThreadLocal in Tika's source, so perhaps it's against your general patterns. Regards, Vilmos You can merge this pull request into a Git repository by running: $ git pull https://github.com/vilmospapp/tika TIKA-1369 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tika/pull/15.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 #15 commit 3a9575fc56a6463b4378b14820e9079352bb1848 Author: Vilmos Papp Date: 2014-07-23T09:18:50Z TIKA-1369 Make SimpleDateFormat usage thread safe --- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---