Hi Lee,

Yes it looks like that parser was not in the service loader in prior
releases and it does not sound like it should be loaded in this way as
it's a ParserDectorator.  I've pushed your PR to main.  I'm still not
entirely sure why this only hits on windows.  Thanks for the contribution!

- Bob

On 8/24/2020 8:05 AM, Peter Lee wrote:
> Hi Bob,
>
> I think I have found out what's wrong. Seems there's a infinite loop. I have 
> pushed a PR, please have a look at :
> https://github.com/apache/tika/pull/343
>
> cheers,
> Lee
>
> On 8 24 2020, at 8:54 , Bob Paulin <[email protected]> wrote:
>> Hi Lee,
>>
>> I get the same error on windows with GeoParser and SentimentAnalysisParser 
>> on the main branch. Removing the Logger fixes both and it builds cleanly. 
>> Still not sure what the exact issue is but I can recreate the issue and your 
>> solution.
>> - Bob
>> On 8/24/2020 4:02 AM, Peter Lee wrote:
>>> Update :
>>>
>>> It works after I removed the loggers in GeoParser and GeoParserConfig. But 
>>> I'm still not clear what exactly the problem is. :(
>>> Lee
>>> On 8 24 2020, at 3:27 , Peter Lee <[email protected]> 
>>> (mailto:[email protected]) wrote:
>>>> Hi all,
>>>>
>>>> The tests are failing on my windows : the GeoParserTest are failing cause 
>>>> the class org.apache.tika.parser.geo.GeoParser cloud not be found. But 
>>>> everything works fine on my Ubuntu.
>>>> The error is wired. I did some googling but couldn't figure out what's the 
>>>> problem.
>>>> Anyone who got same error in Windows?
>>>> Lee
>

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to