You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tika.apache.org by GitBox <gi...@apache.org> on 2020/10/21 13:43:45 UTC

[GitHub] [tika] tballison commented on pull request #369: Use IOException instead of IOExceptionWithCause

tballison commented on pull request #369:
URL: https://github.com/apache/tika/pull/369#issuecomment-713586085


   My memory was that we had agreed to add `commons-io` to tika-core in 2.x, and Bob already did that in the 2.x branch, I haven't had a chance to port those updates to `main` yet. 
   
   I, frankly, want to keep some subclass of IOException around whether that's IOExceptionWithCause or something else.  My reasoning is that we have to wrap SAXExceptions and TikaExceptions in IOExceptions because of the exception signatures in some of our overridden classes.  It is useful to be able differentiate this hack from an actual IOException.
   
   Is there a better way to achieve this goal?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org