4teamwork / ftw.tika

This product integrates Apache Tika for full text indexing with Plone.
4 stars 1 forks source link

Switch to JAXRS server implementation #21

Closed lukasgraf closed 10 years ago

lukasgraf commented 10 years ago

This PR changes the client implementation in ftw.tika to talk to a Tika JAXRS server (HTTP / RESTful) instead of the old builtin server from tika-app.jar (plain TCP).

The old TCP server is not supported by ftw.tika >= 2.0 anymore.

Changes

tika.cfg example / testing buildout

@phgross @maethu please review

/cc @jone

lukasgraf commented 10 years ago

Tests are flaky right now, working on it. I may have to increase the delay for the testrunner to wait for the Tika server to boot up.

lukasgraf commented 10 years ago

@phgross @maethu fixed flaky tests.

phgross commented 10 years ago

Just the detail I've commented above, otherwise looks :+1:

jone commented 10 years ago

@lukasgraf nice! I think we should use another port for the JAXRS implementation in the ftw-buildouts, maybe with a new buildout config file to extend from. When upgrading tika, the buildout URL can be modified. The existing tika.cfg in ftw-buildouts should not be changed to JAXRS so that a rerun with old tika pinnings still works. What do you think?