W00t! Thank you!

On Mon, Aug 24, 2020 at 9:44 AM Bob Paulin <[email protected]> wrote:

> 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]> <[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]> <[email protected]> 
> (mailto:[email protected] <[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
>
>

Reply via email to