dasch-swiss / beol

Bernoulli-Euler OnLine
https://beol.dasch.swiss
GNU Affero General Public License v3.0
0 stars 1 forks source link

Full text search in the connected repositories #147

Closed SepidehAlassi closed 5 years ago

SepidehAlassi commented 5 years ago
SepidehAlassi commented 5 years ago

full text search in the 3rd party repos

SepidehAlassi commented 5 years ago

@flavens Thank you for reviewing this, I asked @tobiasschweizer to check it also regarding the computation time. If it seems critical, I might need to optimise the code a bit.

tobiasschweizer commented 5 years ago

The ontology cache does not work correctly when doing several async search requests because the result is not written back completely before doing new requests for the same ontology. This results in unnecessary calls to the Knora ontology route.

The easiest way to solve this problem would be the caching of all relevant ontologies at the very beginning. But I would really like to look for a better solution so that the ontology cache handles this by itself.

SepidehAlassi commented 5 years ago

@tobiasschweizer can you please review this again? I believe now that the text search is faster, we can safely ignore adding a checkbox

SepidehAlassi commented 5 years ago

@tobiasschweizer I applied your suggestions, can you please check again?