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

Try to add item links within ES query directly. #180

Open
sharkinsspatial opened this issue May 24, 2019 · 1 comment
Open

Try to add item links within ES query directly. #180

sharkinsspatial opened this issue May 24, 2019 · 1 comment

Comments

@sharkinsspatial
Copy link
Contributor

Currently we are manually processing the ES query results at the API level to add the relevant item links. This is inefficient and we should investigate the potential for performing some token replacement internally with the ES query execution for items.

@matthewhanson
Copy link
Member

The drawback here is that this would be require duplicating code in every backend (assuming other non-elasticsearch backends eventually get created).

Do we have an idea of the actual performance hit here?

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

2 participants