Search API V3: Difference between revisions
(Add security note, further API information) |
No edit summary |
||
(11 intermediate revisions by 3 users not shown) | |||
Line 1: | Line 1: | ||
== Overview == | == Overview == | ||
This document serves as a technical proposal for a new search API. | This document serves as a technical proposal for a new search API - V3 ([[Open Food Facts Search API Version 2|see V2 here]]). | ||
'''NOTE''': '''We're now extracting search to [[Search-a-licious]], which has an OpenAPI documentation, an Elastic-Search backend and everything you'd expect from a modern search API.''' | |||
'''IMPORTANT:''' our goal have shifted a bit on this project. See [https://docs.google.com/document/d/1mibE8nACcmen6paSrqT9JQk5VbuvlFUXI1S93yHCK2I/edit Search-a-licious roadmap architecture notes] | |||
=== Goals === | === Goals === | ||
Line 40: | Line 43: | ||
=== Data === | === Data === | ||
The core datatype will be the ''Product''. | The core datatype will be the ''Product''. This data will be stored sparsely (ie, fields that are not set will not be stored in the index) to reduce space. | ||
To enable API cases such as partial text search, a rich autocomplete, and the possibility of eventually serving as a unified read layer, [https://static.openfoodfacts.org/data/data-fields.txt all fields] will be added to the index. Only product names, brands and categories will be indexed for autocomplete queries. | To enable API cases such as partial text search, a rich autocomplete, and the possibility of eventually serving as a unified read layer, [https://static.openfoodfacts.org/data/data-fields.txt all fields] will be added to the index. Only product names, brands and categories will be indexed for autocomplete queries. | ||
Line 74: | Line 77: | ||
* [https://openfoodfacts.github.io/api-documentation/#3SEARCHRequests Require documentation] | * [https://openfoodfacts.github.io/api-documentation/#3SEARCHRequests Require documentation] | ||
* [https://github.com/openfoodfacts/openfoodfacts-server/tree/111e0afdbac3c20ea34652b0b413be58be6dfae5/conf/nginx Need to be exposed in the Nginx configuration] | * [https://github.com/openfoodfacts/openfoodfacts-server/tree/111e0afdbac3c20ea34652b0b413be58be6dfae5/conf/nginx Need to be exposed in the Nginx configuration] | ||
* Be prefixed with ''/v3'' (redirecting at the Nginx layer, but not using the v3 prefix at the Search Service layer) | |||
The proposed API definition is below. Note that the requests are represented as Python objects as used in FastAPI - in reality, this is a JSON payload: | The proposed API definition is below. Note that the requests are represented as Python objects as used in FastAPI - in reality, this is a JSON payload: | ||
Line 91: | Line 95: | ||
field: str | field: str | ||
value: str | value: str | ||
# One of eq, ne, like | # One of eq, ne, like | ||
operator: str = 'eq' | operator: str = 'eq' | ||
Line 98: | Line 102: | ||
field: str | field: str | ||
value: float | value: float | ||
# One of eq, ne, lt, gt | # One of eq, ne, lt, gt | ||
operator: str = 'eq' | operator: str = 'eq' | ||
Line 105: | Line 109: | ||
field: str | field: str | ||
value: datetime.datetime | value: datetime.datetime | ||
# One of | # One of lt, gt | ||
operator: str = 'eq' | operator: str = 'eq' | ||
Line 131: | Line 135: | ||
=== API Discussion === | === API Discussion === | ||
The barcode GET API is included to demonstrate how this service could easily replace our existing APIs, but is not intended to be used for the moment. | The barcode GET API is included to demonstrate how this service could easily replace our existing read APIs, but is not intended to be used for the moment. | ||
The remaining APIs have several commonalities: | The remaining APIs have several commonalities: | ||
* An optional ''response_fields'' parameter is provided, to limit the fields in the response further | * An optional ''response_fields'' parameter is provided, to limit the fields in the response further | ||
* POST is used, to support a complex request body | * POST is used, to support a complex request body | ||
Line 143: | Line 146: | ||
* ''like'' operator, which does not need to match to exact fields, but rather will match by use of the [https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-snowball-tokenfilter.html snowball token filter] in Elasticsearch if the field supports it. | * ''like'' operator, which does not need to match to exact fields, but rather will match by use of the [https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-snowball-tokenfilter.html snowball token filter] in Elasticsearch if the field supports it. | ||
* ''gt'', ''lt'' (greater than, less than) operators are provided. | * ''gt'', ''lt'' (greater than, less than) operators are provided. | ||
* | * There is no support for an OR operator (which is supported in the [[Open Food Facts Search API Version 2|V2 API]]), however clients can perform this logic themselves if they wish through multiple API calls. | ||
=== Relationship With Existing APIs === | |||
The autocomplete API does not overlap with the other APIs. However, the new proposed ''/search'' API has a lot of conceptual overlap with the current search APIs. It is proposed to encourage users to migrate to this new V3 API (and to use it when writing new code) through guidance in the API docs. | |||
Furthermore, once the new API has proven its stability, the legacy ''search.pl'' should be able to be switched over (with a translation layer to map between old and new API request and response norms). The v2 API can also be migrated, but will require a solution to supporting the OR operator (potentially dropping support, using another translation layer with multiple requests, or modifying the API). | |||
This work is considered out of scope for this proposal. | |||
== Work Plan == | == Work Plan == | ||
* Completed | * Completed | ||
** Product document definition | ** Product document definition | ||
** Bulk import script | ** Bulk import script | ||
** API definition | ** API definition | ||
** API implementation | ** API implementation | ||
** Docker, Elasticvue configuration | ** Docker, Elasticvue configuration | ||
** API tests | |||
** Redis Integration | |||
* TODO: | * TODO: | ||
** Deploy (without any traffic) | |||
** Deploy (without any traffic) | |||
** Redis writer on the Perl side | ** Redis writer on the Perl side | ||
** Final testing | ** Final testing | ||
** Document API | ** Document API | ||
** Ongoing monitoring | ** Ongoing monitoring | ||
== Future Work == | |||
There appears to be a lot of opportunities to utilize this infrastructure further: | |||
* Incorporate the autocomplete API in to the main search bar | |||
* Use the new search API to handle searches on the primary website (should result in improved ranking, matching, lower DB load) | |||
* Robotoff could use the same cluster | |||
* Current search APIs could be switched over (with a translation layer) | |||
[[Category:Search]] |
Latest revision as of 09:13, 15 August 2024
Overview
This document serves as a technical proposal for a new search API - V3 (see V2 here). NOTE: We're now extracting search to Search-a-licious, which has an OpenAPI documentation, an Elastic-Search backend and everything you'd expect from a modern search API.
IMPORTANT: our goal have shifted a bit on this project. See Search-a-licious roadmap architecture notes
Goals
Create new search APIs to facilitate:
- Autocomplete (currently unsupported)
- Facet search (replacing our current search API)
- Migrating internal services (such as the search bar) to this API. However, performing this migration is not in scope.
Non Goals
- Improving the current API
- Modifying the current search UX
Elasticsearch
Elasticsearch has been chosen for several reasons:
- Advanced search functionality (easy to support features such as autocomplete)
- Mature ecosystem
- Easy to host, completely free version suits our needs
Configuration
We will use Elasticsearch 8.3.3 (latest), deployed via Docker. We will use a replication factor of one (ie, two copies of the data in total), with shards split across two nodes.
From testing on a M1 Macbook Pro, we see:
- Memory: ~1GB per node
- CPU: Low single digits
- Disk usage: ~6GB after indexing, reduces to ~4GB over time
- Latency: Autocomplete queries take ~0.1s on average
- Import time (full index from CSV): ~15 mins
Given the relatively low resource usage, we could likely use more nodes. For now, it is suggested to monitor resource usage, and add nodes (and increase the replication factor) if needed.
X-Pack security will be turned off, as we do not need to encrypt any communication to Elasticsearch or between the nodes. This simplifies the setup and install of elasticvue (below). If this index was used for more sensitive data in the future, this decision should be revisited.
Monitoring
We will use elasticvue to see information such as resource usage, sharding, and perform debugging. Query stats (to monitor ongoing use) can be seen at elasticvue --> indices --> settings/cog --> Show stats.
Data
The core datatype will be the Product. This data will be stored sparsely (ie, fields that are not set will not be stored in the index) to reduce space.
To enable API cases such as partial text search, a rich autocomplete, and the possibility of eventually serving as a unified read layer, all fields will be added to the index. Only product names, brands and categories will be indexed for autocomplete queries.
An argument could be made for storing fewer fields, and reducing disk usage. However, as illustrated above, disk usage is quite reasonable.
Search Service
The Search Service will be written in Python, using FastAPI. It will leverage Elasticsearch DSL to aid in data modeling and query writing.
Importing Data
A Redis container will be created, which will serve as a queue/buffer for writing data. Using this approach is preferable to a webhook as search service instability will not affect the main server, and we can better handle write spikes/DOS attacks.
Data will be added to the queue when the store_product method is called on the main service. This data will contain the full product definition and a field will indicate if this is an upsert or a delete.
The Search Service will consume from this queue via the Redis Python API, indexing (or deleting) each product as it receives messages.
A manual import script will also be written, to take the CSV file and bulk import items. To ensure that data from the manual import script is up to date (ie, no gap from the time of running the script and when data is imported), we need to:
- Modify the Search Service to perform a set like:
- SET product:<timestamp>:<barcode> <full_product_definition> EX 129600
- Explanation: Set a key for each product that expires in 36 hours
- Modify the import script to:
- See when the last updated/created timestamp was, store as import_cutoff
- Fetch all recent writes from redis with a SCAN, matching anything of the form product:*
- Iterate through timestamps in order that are after import_cutoff
- Apply those writes
API Overview
These APIs will:
- Return a maximum of 100 items
- Require robust unit tests
- Require documentation
- Need to be exposed in the Nginx configuration
- Be prefixed with /v3 (redirecting at the Nginx layer, but not using the v3 prefix at the Search Service layer)
The proposed API definition is below. Note that the requests are represented as Python objects as used in FastAPI - in reality, this is a JSON payload:
class SearchBase(BaseModel): response_fields: Optional[Set[str]] class AutocompleteRequest(SearchBase): text: str search_fields: List[str] = constants.AUTOCOMPLETE_FIELDS class StringFilter(BaseModel): field: str value: str # One of eq, ne, like operator: str = 'eq' class NumericFilter(BaseModel): field: str value: float # One of eq, ne, lt, gt operator: str = 'eq' class DateTimeFilter(BaseModel): field: str value: datetime.datetime # One of lt, gt operator: str = 'eq' class SearchRequest(SearchBase): # Works as an intersection/AND query string_filters: List[StringFilter] numeric_filters: List[NumericFilter] date_time_filters: List[DateTimeFilter]
These are then used as follows:
# Gets the product matching a barcode, included to demonstrate potential usage to replace the main read API, this should not be exposed unless it is decided to send all reads through this service GET /barcode/<barcode> # Autocomplete request POST /autocomplete, body=AutocompleteRequest # Fully fledged search POST /search, body=SearchRequest
API Discussion
The barcode GET API is included to demonstrate how this service could easily replace our existing read APIs, but is not intended to be used for the moment.
The remaining APIs have several commonalities:
- An optional response_fields parameter is provided, to limit the fields in the response further
- POST is used, to support a complex request body
The /search API is the most complex as it allows a series of filters to support the use cases in the current API. These filters will work like an intersection/AND query. Of interest are the:
- like operator, which does not need to match to exact fields, but rather will match by use of the snowball token filter in Elasticsearch if the field supports it.
- gt, lt (greater than, less than) operators are provided.
- There is no support for an OR operator (which is supported in the V2 API), however clients can perform this logic themselves if they wish through multiple API calls.
Relationship With Existing APIs
The autocomplete API does not overlap with the other APIs. However, the new proposed /search API has a lot of conceptual overlap with the current search APIs. It is proposed to encourage users to migrate to this new V3 API (and to use it when writing new code) through guidance in the API docs.
Furthermore, once the new API has proven its stability, the legacy search.pl should be able to be switched over (with a translation layer to map between old and new API request and response norms). The v2 API can also be migrated, but will require a solution to supporting the OR operator (potentially dropping support, using another translation layer with multiple requests, or modifying the API).
This work is considered out of scope for this proposal.
Work Plan
- Completed
- Product document definition
- Bulk import script
- API definition
- API implementation
- Docker, Elasticvue configuration
- API tests
- Redis Integration
- TODO:
- Deploy (without any traffic)
- Redis writer on the Perl side
- Final testing
- Document API
- Ongoing monitoring
Future Work
There appears to be a lot of opportunities to utilize this infrastructure further:
- Incorporate the autocomplete API in to the main search bar
- Use the new search API to handle searches on the primary website (should result in improved ranking, matching, lower DB load)
- Robotoff could use the same cluster
- Current search APIs could be switched over (with a translation layer)