Github user osma commented on the issue:
https://github.com/apache/jena/pull/227
Hi @anujgandharv, thanks for the update, I will do a new review soon.
Regarding `TextIndex.get`, excellent question! I was actually wondering the
same when I did the review yesterday, but didn't take a closer look then. It
seems like this method is never called from within Jena! The only exception is
your TestTextIndexES class that specifically tests for this method.
I don't know what the reason for this method is. Maybe it was useful some
time ago, or maybe it was created for some purpose that never really
materialized. I think the `get` method could simply be removed from the
TextIndex interface and all implementations. I'm sorry that you had to spend
time implementing it.
The situation is similar for the `updateEntity` method. It is also not
called from Jena code and could be dropped.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
