Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make character limit for (Tika) BodyContentHandler a config option #4

Open
straup opened this issue May 15, 2014 · 1 comment
Open

Comments

@straup
Copy link
Collaborator

straup commented May 15, 2014

See also:

http://stackoverflow.com/questions/6144708/apache-tika-and-character-limit-when-parsing-documents

Where do config flags go anyway?

@straup
Copy link
Collaborator Author

straup commented May 15, 2014

And then he said:

I am with you all the way up to the very last yaml file :) Presumably the dropwizard Configuration class provides methods to read properties set in the yaml file? Looks like you add some annotations and getters/setters for your property to the ExtruderConfiguration class and then set them as such:

http://grepcode.com/file/repo1.maven.org/maven2/com.yammer.dropwizard/dropwizard-core/0.1.3/com/yammer/dropwizard/config/Configuration.java

First I said:

Can you confirm or deny a couple Dropwizard / Java things for me?

I am trying to fix this issue:

https://github.com/straup/dogeared-extruder/issues/4

Which basically means setting a value here, in the BodyContentHandler constructor:

https://github.com/straup/dogeared-extruder/blob/master/src/main/java/info/aaronland/extruder/TikaResource.java#L154

Which I think means I need to pass "ExtruderConfiguration conf" to the TikaResource constructor here:

https://github.com/straup/dogeared-extruder/blob/master/src/main/java/info/aaronland/extruder/ExtruderService.java#L24-L33

Which I am pretty sure is this:

https://github.com/straup/dogeared-extruder/blob/master/src/main/java/info/aaronland/extruder/ExtruderConfiguration.java

Which I am less sure is this:

https://github.com/straup/dogeared-extruder/blob/master/configuration.yml

Does that sound about right? Because I only ever write Java with 8 month breaks in between I am never entirely confident of my hunches and because Java is full of debugging quicksand I am always a bit nervous to disturb a thing that already works...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant