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

Compatibility with icat.lucene upgrades #267

Open
patrick-austin opened this issue Jan 28, 2022 · 2 comments · May be fixed by #298
Open

Compatibility with icat.lucene upgrades #267

patrick-austin opened this issue Jan 28, 2022 · 2 comments · May be fixed by #298

Comments

@patrick-austin
Copy link
Contributor

Description:
In the process of making changes to icat.lucene to improve search functionality, the interface in LuceneAPI, LuceneManager etc. will become incompatible. For now these changes are not finalised, but to test them out as part of the full stack need to make the changes to icat.server.

For detail, see icatproject/icat.lucene#18 which also links to sub-issues.

@RKrahl
Copy link
Member

RKrahl commented Jan 28, 2022

Will these changes be compatible for clients? E.g. will the exposed ICAT API calls for searching or for interacting with the lucene db change?

@patrick-austin
Copy link
Contributor Author

Will these changes be compatible for clients? E.g. will the exposed ICAT API calls for searching or for interacting with the lucene db change?

Currently neither of those would be affected by the changes, at the moment it's things like classes being renamed in more recent versions of Lucene (which are needed for some functionality).

Going forward there might be changes such as allowing different entities or fields to be searched with Lucene (as currently we only search on Investigations, Datasets/files with Samples/Parameters). But at this stage it's somewhat exploratory to see how additional functionality might be implemented with Lucene.

patrick-austin added a commit that referenced this issue Apr 6, 2022
patrick-austin added a commit that referenced this issue Apr 19, 2022
patrick-austin added a commit that referenced this issue Apr 20, 2022
patrick-austin added a commit that referenced this issue Jun 16, 2022
patrick-austin added a commit that referenced this issue Aug 9, 2022
patrick-austin added a commit that referenced this issue Aug 18, 2022
patrick-austin added a commit that referenced this issue Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants