This repository has been archived by the owner on Mar 21, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 11
Tasks API - Add task deletion capability #198
Merged
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
v0.30
Ready For Review
Feature specification must be reviewed.
OpenAPI
Update OpenAPI specification.
Q4:2022
Telemetry
Update the telemetry collect.
labels
Nov 2, 2022
gmourier
changed the title
Tasks API - Add task deletion feature
Tasks API - Add task deletion capability
Nov 2, 2022
This was referenced Nov 2, 2022
Merged
4 tasks
gmourier
force-pushed
the
delete-tasks-feature
branch
from
November 2, 2022 17:40
a041a40
to
fbf6753
Compare
loiclec
reviewed
Nov 3, 2022
loiclec
reviewed
Nov 3, 2022
gmourier
force-pushed
the
delete-tasks-feature
branch
3 times, most recently
from
November 3, 2022 15:47
e8c95b7
to
9837329
Compare
gmourier
force-pushed
the
cancel-tasks-feature
branch
from
November 3, 2022 17:16
666ae60
to
3403292
Compare
gmourier
force-pushed
the
delete-tasks-feature
branch
from
November 3, 2022 17:28
9837329
to
28475c9
Compare
23 tasks
gmourier
force-pushed
the
delete-tasks-feature
branch
from
November 3, 2022 21:26
28475c9
to
70c4b4b
Compare
13 tasks
Kerollmops
approved these changes
Nov 15, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
5 tasks
3 tasks
Kerollmops
suggested changes
Nov 17, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to our discussion on Slack, we must go with originalFilter
because, technically, it is one filter expression and for consistency with the search route.
gmourier
force-pushed
the
delete-tasks-feature
branch
from
November 21, 2022 13:33
11503f3
to
8fff245
Compare
4 tasks
Co-authored-by: Clément Renault <renault.cle@gmail.com>
gmourier
force-pushed
the
delete-tasks-feature
branch
from
November 23, 2022 14:04
8fff245
to
1fb8f8a
Compare
Kerollmops
approved these changes
Nov 28, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much! I like what you've done!
gmourier
added
Implemented
Feature specification has been implemented.
and removed
Ready For Review
Feature specification must be reviewed.
labels
Nov 29, 2022
gmourier
added a commit
that referenced
this pull request
Nov 29, 2022
* Add cancel to task spec * Remove missing error * Fix typo on cancels * Fix IndexUid always null * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Add canceledBy and missing_filters error * Merge 11.1 filter iwth get and 11.2 filter with post * Specified every date filter * Change uid as integer and not as string * Add auto-batching and error * Add new route task/cancel api-key * Add global task note * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0061-error-format-and-definitions.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Modification due to review * Modification due to review * Modification due to review * Modification due to review * Fix global task link * Fix format link * Add open-api modification * Remove trailing space * Add clarification * Add errors definition * Update open-api.yml * fix task uid query parameter name * Fix collission between url path and query parameters * fmt open-api.yml * Add telemetry details * Add 202_taskCancelation example * Update originalQuery to originalFilters and update missing_task_filter to missing_task_filters * matchedTasks is computed when the task is enqueued * Apply suggestions from code review Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> * Recall explanation for filter query parameter and mentions that limit and from are valid on * Fix typo in property name * Update query paramters name and related error codes when invalid * Fix missing part of error msg * Precise valid date formats * Add supported format for date filters in open-api.yml * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0060-tasks-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> * fix typo * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0060-tasks-api.md * Update text/0060-tasks-api.md * Tasks API - Add task deletion capability (#198) * Describes task deletion feature additions * Update open-api.yml * Update telemetry-policies * Update originalQuery to originalFilters * Follow title structure * Introduce review feedback * Branch the right components * Update text/0060-tasks-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> Co-authored-by: Clément Renault <renault.cle@gmail.com>
gmourier
added a commit
that referenced
this pull request
Nov 29, 2022
* Bump Meilisearch OpenAPI spec to v0.30.0 * Compression Headers (#178) * Start specification * Rename spec file * Rephrase summary * Rephrase receiving a compressed response section * Add links to RFC9910 * Introduce missing_master_key error (#179) * Update error message (#180) * Specify the error variant (#181) * replaces 127.0.0.1 by localhost (#184) * Configuration File Support (#185) * init spec file * Fill-in spec * Fix typing errors * adds errors case * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Replace config file by configuration file in the error message * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Add infos.with_configuration_file to telemetry-policies * Update text/0185-configuration-file.md Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Tamo <tamo@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Tamo <tamo@meilisearch.com> * Update text/0185-configuration-file.md Co-authored-by: Tamo <tamo@meilisearch.com> Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> Co-authored-by: Tamo <tamo@meilisearch.com> * Telemetry - Missing events and options (#186) * wip spec changes * update telemetry * add missing property to the full properties table * Update text/0034-telemetry-policies.md Co-authored-by: Tamo <tamo@meilisearch.com> * Update text/0034-telemetry-policies.md Co-authored-by: Tamo <tamo@meilisearch.com> * Add distrinct_attribute.set property, Task Seen becomes an aggregated event, add missing sub settings properties to Settings Updated event * Add requests.total_received property to Tasks Seen event * add missing ranking rules metrics * Update text/0034-telemetry-policies.md Co-authored-by: Tamo <tamo@meilisearch.com> * Update text/0034-telemetry-policies.md Co-authored-by: Tamo <tamo@meilisearch.com> * Update text/0034-telemetry-policies.md Co-authored-by: Tamo <tamo@meilisearch.com> * Precise max-index-size option (#187) * Tasks API - Rename `receivedDocumentsIds` by `providedIds` (#188) * Rename receivedDocumentsIds by matchedDocuments * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0060-tasks-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> * Add missing env vars, brings consistency (#190) * Swap Indexes API (#192) * Add new spec file as draft file * Add specification file * update spec * draft open-api.yml changes * Add errors * fix typos * Add Indexes Swapped event * duplicate_index_found is synchronous * Apply suggestions from code review Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> * Add review suggestions * Precis mention for tasks indexUid target swap in the history * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0061-error-format-and-definitions.md Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0191-swap-indexes-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> Co-authored-by: Clément Renault <renault.cle@gmail.com> * Add snapshotCreation as a new task type to make it visible in the task history (#193) * Tasks API - Add task priority order section (#194) * Add 2.1 Task Priority section * Precise other tasks priority ordering * Search API - Page Selection Component (#196) * Search API - numbered pagination * Improve numbered pagination explaination * Update text/0118-search-api.md * Update text/0118-search-api.md * Apply suggestions from code review * Add links * Update open api * Add a dot at the end of the title * Fix links * Improve page and hitsperpage query params * Refactor pagination guide * Fix wording * Add corner case * Update open-api.yaml Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update open-api.yaml Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update open-api.yaml Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Apply wording changes from code review Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Apply suggestions from code review Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com> * Fix example * Apply suggestions from code review Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update open-api.yaml * Update text/0118-search-api.md * Update links * Add link to maxtotalhits * Update text/0118-search-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Improve message in page reference * Update text/0118-search-api.md Co-authored-by: Many the fish <legendre.maxime.isn@gmail.com> * Add telemetry to know the most used search results navigation methods in a batch * Update text/0118-search-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0118-search-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com> Co-authored-by: Many the fish <legendre.maxime.isn@gmail.com> * Tasks API - Add task cancelation capability (#195) * Add cancel to task spec * Remove missing error * Fix typo on cancels * Fix IndexUid always null * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Add canceledBy and missing_filters error * Merge 11.1 filter iwth get and 11.2 filter with post * Specified every date filter * Change uid as integer and not as string * Add auto-batching and error * Add new route task/cancel api-key * Add global task note * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0060-tasks-api.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Update text/0061-error-format-and-definitions.md Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> * Modification due to review * Modification due to review * Modification due to review * Modification due to review * Fix global task link * Fix format link * Add open-api modification * Remove trailing space * Add clarification * Add errors definition * Update open-api.yml * fix task uid query parameter name * Fix collission between url path and query parameters * fmt open-api.yml * Add telemetry details * Add 202_taskCancelation example * Update originalQuery to originalFilters and update missing_task_filter to missing_task_filters * matchedTasks is computed when the task is enqueued * Apply suggestions from code review Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> * Recall explanation for filter query parameter and mentions that limit and from are valid on * Fix typo in property name * Update query paramters name and related error codes when invalid * Fix missing part of error msg * Precise valid date formats * Add supported format for date filters in open-api.yml * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0060-tasks-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> * fix typo * Update open-api.yaml Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update text/0060-tasks-api.md * Update text/0060-tasks-api.md * Tasks API - Add task deletion capability (#198) * Describes task deletion feature additions * Update open-api.yml * Update telemetry-policies * Update originalQuery to originalFilters * Follow title structure * Introduce review feedback * Branch the right components * Update text/0060-tasks-api.md Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com> Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> Co-authored-by: Clément Renault <renault.cle@gmail.com> * Update the dump support table with the dumpv6 (#199) * Settings API - Specify handling of null values when patching settings (#202) * Specify behavior when patching a setting with partial values and null values. - Partial value: the setting is unchanged. - null value: the setting is reset to default. * Settings: Update error messages to accept the null value * Add missing task types * Add canceledTasks details Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com> Co-authored-by: Tamo <tamo@meilisearch.com> Co-authored-by: Clément Renault <renault.cle@gmail.com> Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com> Co-authored-by: Bruno Casali <brunoocasali@gmail.com> Co-authored-by: Many the fish <legendre.maxime.isn@gmail.com> Co-authored-by: Amélie <alallema@users.noreply.github.com> Co-authored-by: Louis Dureuil <louis.dureuil@gmail.com>
3 tasks
2 tasks
11 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Implemented
Feature specification has been implemented.
OpenAPI
Update OpenAPI specification.
Q4:2022
Telemetry
Update the telemetry collect.
v0.30
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 the
OpenApi
labelSummary
Add the
DELETE /tasks
API endpointChanges
/tasks
taskDeletion
tasks.delete
Tasks Deleted
telemetry eventuser_agent
filtered_by_uid
filtered_by_index_uid
filtered_by_status
filtered_by_type
filtered_by_canceled_by
filtered_by_before_enqueued_at
filtered_by_after_enqueued_at
filtered_by_before_started_at
filtered_by_after_started_at
filtered_by_before_finished_at
filtered_by_after_finished_at
Out Of Scope
n/a
Attention To Reviewers
n/a
Misc