OPENNLP-936: Add thread-safe versions of POSTaggerME, SentenceDetecto…
…rME and TokenizerME. Include test case as well.
I'm open to changing the names of the classes, if anybody has a better idea.
Thanks!
- Indent by 2 spaces (not 4), checkstyle is giving many warnings about that (click the green check mark to see the travis-ci output)
- Each new file needs to have AL 2.0 license header
- Commit messages subject line is limited to 72 chars, please make it short and also use the body of it
- We strongly recommend to set up Eclipse or Intellij with the style files from our code convention page (https://opennlp.apache.org/code-conventions.html)
- Update the commit above e.g. via git commit --amend or squash and then force push (git push -f) to update the PR
Ok done.
This issue (OPENNLP-936) is still relevant to many users. Is there any form of consensus - in 2022 - on how to progress with the current code base?
The comments on this PR seem fairly trivial. I have no objections to getting those comments addressed and merging this. I will go through the PR again and see if there's anything since then that should be addressed.
Thanks @mawiesne for bringing attention to this one.