[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-06-30 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/oodt/pull/61 @IMS94 can you take a look at this? ---

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-23 Thread IMS94
Github user IMS94 commented on the issue: https://github.com/apache/oodt/pull/61 @chrismattmann Sure. Will look at this. ---

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-23 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/oodt/pull/61 @IMS94 can you take a look at this? Do tests pass with the upgrade? ---

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-18 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/oodt/pull/61 @lewismc can't you simply monkey patch DRAT to use the new Tika transitively that OODT would install should this PR be merged, and test it and report back? If I get time today I'll try that myself

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-01 Thread lewismc
Github user lewismc commented on the issue: https://github.com/apache/oodt/pull/61 @chrismattmann there is an issue in testing this over in DRAT. OODT 1.2.2 modules use Tika 1.10 and 1.7 dependencies... hence DRAT distribution packages a deployment which inherits the older Tika dep

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-01 Thread lewismc
Github user lewismc commented on the issue: https://github.com/apache/oodt/pull/61 @chrismattmann https://github.com/apache/tika/pull/235 ---

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-01 Thread lewismc
Github user lewismc commented on the issue: https://github.com/apache/oodt/pull/61 > ...I am not sure we need to rename it since I believe it is explicitly identified in the bin/filemgr script as an argument or system property, e.g., here. ACK > ...Also can you subm

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-01 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/oodt/pull/61 @lewismc great question. I am not sure we need to rename it since I believe it is explicitly identified in the bin/filemgr script as an argument or system property, e.g., [here](https://github.co

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-05-01 Thread lewismc
Github user lewismc commented on the issue: https://github.com/apache/oodt/pull/61 Hi @chrismattmann I'm testing with DRAT... quick question though. Is it necessary for us to rename the ```mime-types.xml``` files present in OODT (and updated as part of the PR) to the Tika equivalent e

[GitHub] oodt issue #61: OODT-976 Upgrade Apache Tika to 1.18

2018-04-25 Thread chrismattmann
Github user chrismattmann commented on the issue: https://github.com/apache/oodt/pull/61 @lewismc can you test with DRAT? my worry is that some of our unit tests miss the full end to end coverage of DRAT. ---