Jukka Zitting wrote:
>> 2) In ParserFactory, we have:
>>
>> } catch (Exception e) {
>>     logger.error("Unable to instantiate parser: " + className, e);
>>     throw new TikaException(e.getMessage());
>> }
>>
>> When we adapt an exception to a TikaException, would it make sense to wrap
>> the entire exception, and not just its getMessage()?
> 
> +1

And I think we should not log the exception in this case. Otherwise the
exception is logged more than once.

Carsten


-- 
Carsten Ziegeler
[EMAIL PROTECTED]

Reply via email to