Skip to content
This repository was archived by the owner on Mar 21, 2024. It is now read-only.

Telemetry - Add Stats Seen event #169

Merged
merged 1 commit into from
Oct 3, 2022

Conversation

gmourier
Copy link
Member

@gmourier gmourier commented Aug 17, 2022

Summary

Add Stats Seen event to the telemetry policies specification.


Changes

  • Add Stats Seen event
    • user_agent: Represents the user-agent encountered on this call
    • per_index_uid: true if an uid is used to fetch an index stat resource, otherwise false

Out Of Scope

n/a


Attention To Reviewers

n/a


Misc

  • Update telemetry datapoints

@gmourier gmourier requested a review from irevoire August 17, 2022 10:24
@gmourier gmourier added Q3:2022 v0.29 Telemetry Update the telemetry collect. Ready For Review Feature specification must be reviewed. labels Aug 17, 2022
@gmourier gmourier mentioned this pull request Aug 17, 2022
1 task
bors bot added a commit to meilisearch/meilisearch that referenced this pull request Aug 18, 2022
2674: Add analytics on the stats routes r=ManyTheFish a=irevoire

# Pull Request

## What does this PR do?
Implements meilisearch/specifications#169

## PR checklist
Please check if your PR fulfills the following requirements:
- [ ] Does this PR fix an existing issue?
- [ ] Have you read the contributing guidelines?
- [ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Irevoire <tamo@meilisearch.com>
@gmourier gmourier requested review from irevoire and removed request for irevoire October 3, 2022 12:08
@gmourier gmourier added Implemented Feature specification has been implemented. and removed Ready For Review Feature specification must be reviewed. labels Oct 3, 2022
@gmourier gmourier merged commit 691c71f into develop Oct 3, 2022
@gmourier gmourier deleted the add-telemetry-event-on-stats-endpoint branch October 3, 2022 14:51
gmourier added a commit that referenced this pull request Oct 3, 2022
gmourier added a commit that referenced this pull request Oct 3, 2022
* Bump openapi spec version to v0.29

* Update 0001-script-based-tokenizer.md (#159)

Change tokenizer specs to better fit Charabia implementation

* Update the geosearch error (#161)

Implemented in meilisearch/milli#561

* Auto-batching - Enable feature by default and remove unwanted options (#162)

* Update specs according to new auto-batching behavior

* update batchUid to make it internal and hidden from a task resource representation

* Remove the batchUid mentions from the task API

* Update open-api.yaml

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* update future possibilities

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Search API — Filters - Introduce IN and EXISTS and describe filter capabilities in more precisely (#163)

* Write a specification for the new (and old) search filters

EXISTS
IN
NOT (new behaviour)
!= (new behaviour)

* Apply suggestions from code review

Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Add missing settings object in the task details field of a settingsUpdate task type (#164)

* Remove `name` from indexes resource definition (#165)

* Misc — Soft deleted documents (Performance optimization) (#168)

* create a spec for the soft deleted documents

* Rename spec file, minor adjustements

* Replace You and We by A user and Meilisearch

Co-authored-by: Guillaume Mourier <guillaume@meilisearch.com>

* Add Stats Seen event (#169)

* Add examples component for each summarized task type (#170)

* Version API — Catch up (#171)

* Add version-api.md

* Add PR number as a spec file prefix

* Add health-api.md (#172)

* Search API — Add `matchingStrategy` parameter with `last` / `all` strategies (#173)

* Introduce a proposal to boot the specification

* Update telemetry

* Replace wordMatchingStrategy by matchingStrategy

* fix missing backtick md

Co-authored-by: Many the fish <legendre.maxime.isn@gmail.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
Co-authored-by: Clémentine Urquizar - curqui <clementine@meilisearch.com>
Co-authored-by: Loïc Lecrenier <loic.lecrenier@me.com>
Co-authored-by: cvermand <33010418+bidoubiwa@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Implemented Feature specification has been implemented. Q3:2022 Telemetry Update the telemetry collect. v0.29
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant