Skip to content

fix(client): downgrade slow query diagnostics#10884

Merged
ArtyomSavchenko merged 1 commit into
hcengineering:developfrom
ignatremizov:fix/client-slow-query-warning
May 26, 2026
Merged

fix(client): downgrade slow query diagnostics#10884
ArtyomSavchenko merged 1 commit into
hcengineering:developfrom
ignatremizov:fix/client-slow-query-warning

Conversation

@ignatremizov
Copy link
Copy Markdown
Contributor

@ignatremizov ignatremizov commented May 25, 2026

Summary

  • Change slow client findAll measurements from console.error to console.warn.
  • Keep the existing slow-query thresholds, log message, payload, and timing details unchanged.
  • Clarify that successful-but-slow client queries are performance diagnostics rather than query errors.

Testing

  • git diff --check upstream/develop...HEAD
  • git diff --check
  • node common/scripts/install-run-rush.js build --to @hcengineering/client-resources

Notes

  • This only changes the browser console severity for slow-query diagnostics. It does not change query execution, thresholds, or diagnostic payloads.

Report slow client findAll measurements as warnings instead of errors so the browser console reflects that these entries are performance diagnostics, not failed queries.

Preserve the existing threshold, payload, and timing details for performance triage.

Signed-off-by: Ignat Remizov <ignat@ignatremizov.com>
@huly-github-staging
Copy link
Copy Markdown

Connected to Huly®: UBERF-16474

@ignatremizov
Copy link
Copy Markdown
Contributor Author

ignatremizov commented May 25, 2026

It appeared like a lot of errors first time, when it's actually just diagnostic noise. Helpful to add indexes from the diagnostics though, so useful to keep the log line.

@ArtyomSavchenko ArtyomSavchenko merged commit 2c151ea into hcengineering:develop May 26, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants