This repository has been archived by the owner on Mar 21, 2024. It is now read-only.
Search Algorithms Refactor: Relevancy Changes #233
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gmourier
added
Catch-Up
This was referenced May 3, 2023
Merged
This was referenced May 4, 2023
|
@gmourier, it looks good to me |
|
Looks good to me, thanks @gmourier |
gmourier
added
Ready For Review
macraig
approved these changes
Jun 5, 2023
macraig
pushed a commit
that referenced
this pull request
Jun 5, 2023
* Bump OpenAPI version * CSV Content-Type — Add support for the boolean type in CSV (#228) * add support for the boolean type in csv * fix a typo * Update text/0028-indexing-csv.md --------- Co-authored-by: Guillaume Mourier <[email protected]> * Filter operators - `IS NULL` and `IS EMPTY` filter operators (#232) * Introduce the spec of the IS EMPTY filter * Introduce the spec of the IS NULL filter * Fix suggestions * Uppercase the IN operator Co-authored-by: Guillaume Mourier <[email protected]> * Fix a title number issue Co-authored-by: Guillaume Mourier <[email protected]> --------- Co-authored-by: Guillaume Mourier <[email protected]> * Search Algorithms Refactor: Relevancy Changes (#233) * Remove old exactness-criterion specification * Remove old asc-desc-criterion specification * Get and delete documents by filter (#236) * specify the new fetch documents route * update open-api * try to be nice with bumpsh * specify the new analytics of the get/fetch documents route * group the analytics behind already used groups * fix a placeholder text * group the get and fetch route under a common section * rename the section to contain the name of both routes * clarify how to send the parameter to both routes * fix the type of the parameters * add the new fetch route to the api keys actions * capitalize json Co-authored-by: cvermand <[email protected]> * update open-api * Specify the new route to delete documents by filter * add the new analytics event * improve a comment * add the post - delete route to the api key actions * Fix the json payload Co-authored-by: cvermand <[email protected]> * Apply suggestions from code review Co-authored-by: Guillaume Mourier <[email protected]> * Update the details Co-authored-by: Guillaume Mourier <[email protected]> * delete the invalid_document_delete_filter error code and introduce the missing_document_filter * fix the open-api thingy --------- Co-authored-by: cvermand <[email protected]> Co-authored-by: Guillaume Mourier <[email protected]> * Auto deletion of tasks (#238) * auto deletion of tasks * Update text/0060-tasks-api.md Co-authored-by: gui machiavelli <[email protected]> * Update text/0060-tasks-api.md Co-authored-by: gui machiavelli <[email protected]> --------- Co-authored-by: gui machiavelli <[email protected]> * Experimental reduce indexing memory usage (#239) * Add the experimental-reduce-indexing-memory-usage option in the telemetry policies * Add the experimental-reduce-indexing-memory-usage option in the instance options * Update 0055-sort.md (#240) related to meilisearch/meilisearch#3749 * Update `payload_too_large` error `message` (#241) * Update 0061-error-format-and-definitions.md Update error message in the specification * Update open-api.yaml --------- Co-authored-by: Tamo <[email protected]> Co-authored-by: Clément Renault <[email protected]> Co-authored-by: cvermand <[email protected]> Co-authored-by: gui machiavelli <[email protected]> Co-authored-by: Many the fish <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤖 API Diff Put the link of the GitHub comment generated by bump.sh if generated; Apply theOpenApilabelSummary
Changes
Cleaning
0036-exactness-criterion.mdspecification - Ranking rules have already been described in0123-ranking-rules-setting-api.md.0030-asc-desc-criterion.mdspecification - Ranking rules have already been described in0123-ranking-rules-setting-api.md.Addition/Update
Out Of Scope
N/A
Attention To Reviewers
N/A
Misc
[] Update OpenAPI specification file (if needed; Apply theOpenApilabel)[] Update telemetry datapoints (if needed; Apply theTelemetrylabel)