Log slow queries in tests
Open
Nobody has claimed this yet.
enhancement
technical debt
- Dominant language
- PHP
- Stars
- 2.2k
- Forks
- 587
- Avg merge
- 18h 27m
- Merged PRs (30d)
- 333
Description
How to use GitHub
- Please use the 👍 reaction to show that you are interested into the same feature.
- Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
- Subscribe to receive notifications on status change and new comments.
Similar to what the mail app did:
https://github.com/nextcloud/mail/pull/6049/files
I guess most useful would be integration tests at first. Although (as per comment in the mail app) because some tables contain too few entries all of the queries will not use an index and therefor show up there.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the referenced nextcloud/mail pull request and the repository's existing integration-test setup. Define how slow-query logging should appear in those tests, including the noted behavior of queries on small tables, and consider the work done when integration tests provide the requested logging.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- testing
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100