Hi Jukka, My +1 for option A and B, I think it's better in the long run since configuration is an issue. All methods from option B should load default static configuration. I'd be happy to take the lead on JIRA'ing and implementing this in 0.5...
Cheers, Chris On 11/11/09 11:21 AM, "Jukka Zitting" <jukka.zitt...@gmail.com> wrote: Hi, At first I used static methods in the Tika facade class, but then I switched to normal instance methods to make it easier to use custom Tika configuration. However, now I'm thinking that the original design was probably better. Any opinions on this? Essentially, which one of the following do you like better: a) new Tika().parse(...); b) Tika.parse(...); The benefit of a) over b) is that you can say: new Tika(config).parse(...), but most of the people who need custom configuration will probably in any case be going beyond the Tika facade. BR, Jukka Zitting ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Chris Mattmann, Ph.D. Senior Computer Scientist NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA Office: 171-266B, Mailstop: 171-246 Email: chris.mattm...@jpl.nasa.gov WWW: http://sunset.usc.edu/~mattmann/ ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Adjunct Assistant Professor, Computer Science Department University of Southern California, Los Angeles, CA 90089 USA ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++