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.

Sign up for GitHub

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

Jump to bottom

v0.28: Telemetry updates #1693

Closed
guimachiavelli opened this issue Jun 7, 2022 · 1 comment · Fixed by #1707
Closed

v0.28: Telemetry updates #1693

guimachiavelli opened this issue Jun 7, 2022 · 1 comment · Fixed by #1707
Assignees
@maryamsulemani97
Milestone
v0.28

Comments

@guimachiavelli
Copy link
Member

guimachiavelli commented Jun 7, 2022 •
edited
Loading

This issue groups all v0.28 updates to the telemetry table:

  • Add a facets telemetry node
  • Rename formatting.matches to formatting.show_matches_position
  • Add new pagination.maxTotalHits node
  • Add new faceting.max_values_per_facet node
  • Add new /task-filtering-related nodes: per_task_uid, filtered_by_index_uid, filtered_by_type, filtered_by_status

References
@guimachiavelli guimachiavelli added this to the v0.28 milestone Jun 7, 2022
@guimachiavelli guimachiavelli self-assigned this Jun 9, 2022
@guimachiavelli guimachiavelli mentioned this issue Jun 15, 2022
Merged
@guimachiavelli guimachiavelli linked a pull request Jun 15, 2022 that will close this issue
v0.28 #1707
Merged
@dichotommy
Copy link
Contributor

dichotommy commented Jun 20, 2022 •
edited
Loading

There are some new telemetry metrics as part of the changes to the tasks API:

| `per_task_uid`          | `true` if an uid is used to fetch a particular task resource, otherwise `false` | true |
| `filtered_by_index_uid` | `true` if `GET /tasks` endpoint is filered by `indexUid`, otherwise `false` | false |
| `filtered_by_type`      | `true` if `GET /tasks` endpoint is filered by `type`, otherwise `false` | false |
| `filtered_by_status`    | `true` if `GET /tasks` endpoint is filered by `status`, otherwise `false` | false |

@maryamsulemani97 maryamsulemani97 mentioned this issue Jun 29, 2022
Merged
bors bot added a commit that referenced this issue Jul 5, 2022
@bors @maryamsulemani97
Merge #1763
52dcfd0 
1763: v0.28: Telemetry updates r=maryamsulemani97 a=maryamsulemani97

closes #1693 

Co-authored-by: Maryam Sulemani <maryam@meilisearch.com>
Co-authored-by: Maryam <90181761+maryamsulemani97@users.noreply.github.com>
bors bot added a commit that referenced this issue Jul 11, 2022
@bors @guimachiavelli
@dichotommy @maryamsulemani97
Merge #1707
00cb6df 
1707: v0.28 r=guimachiavelli a=guimachiavelli

This is a staging PR for all changes related to Meilisearch v0.28.

Please avoid making changes directly to this PR; instead, create new child branches based off this one.

Closes #1687, #1688, #1691, #1692, #1693, #1694, #1699, #1700, #1701, #1702, #1703, #1704, #1706, #1722, #1727, #561

Co-authored-by: gui machiavelli <hey@guimachiavelli.com>
Co-authored-by: gui machiavelli <gui@meilisearch.com>
Co-authored-by: Tommy Melvin <tommy@meilisearch.com>
Co-authored-by: Maryam Sulemani <maryam@meilisearch.com>
Co-authored-by: Maryam <90181761+maryamsulemani97@users.noreply.github.com>
@bors bors bot closed this as completed in #1707 Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maryamsulemani97 maryamsulemani97

Labels
None yet
Projects
None yet
Milestone
v0.28
Development

Successfully merging a pull request may close this issue.

v0.28 meilisearch/documentation
3 participants
@guimachiavelli @dichotommy @maryamsulemani97