tballison commented on PR #2118:
URL: https://github.com/apache/stormcrawler/pull/2118#issuecomment-5532258421

   Other subtly, and we should have a unit test for this is that 
   
   ```
           if (extractEmbedded) {
               parseContext.set(Parser.class, tika.getParser());
           }
   ```
   
   I think we should invert this to put in the `EmptyParser` if `! 
extractEmbedded` because the AutoDetectParser adds itself (and did in 3.x, I 
thought). So, I think this was already broken?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to