Improvement: MetricsExplorer Module suggested improvements

[nikhilmantri0902]

📄 Summary

#7791 (comment)

---

✅ Changes

• Feature: Brief description
• Bug fix: Brief description

---

🏷️ Required: Add Relevant Labels

⚠️ Manually add appropriate labels in the PR sidebar
Please select one or more labels (as applicable):

ex:

• frontend
• backend
• devops
• bug
• enhancement
• ui
• test

---

👥 Reviewers

Tag the relevant teams for review:

• frontend / backend / devops

---

🧪 How to Test

1. ...
2. ...
3. ...

---

🔍 Related Issues

Closes #

---

📸 Screenshots / Screen Recording (if applicable / mandatory for UI related changes)

---

📋 Checklist

• Dev Review
• Test cases added (Unit/ Integration / E2E)
• Manually tested the changes

---

👀 Notes for Reviewers

---

Note

Introduces MetricsExplorer config (controls ClickHouse max_threads), renames treemap request field to mode, sets thread limits on queries, defaults stats ordering by samples, uses GLOBAL IN/anyLast, and requires edit access for metric metadata updates.

• Metrics Explorer:
  • Config: New metricsexplorer config with telemetrystore.threads; wired through app init and tests.
  • Query execution: Applies max_threads via context on all ClickHouse queries/inserts.
  • Stats defaults: GetStats now defaults order-by to samples when unspecified.
  • Treemap API: Request field renamed to mode (from treemap); switch logic updated accordingly.
  • SQL tweaks: Use anyLast for metadata selection; use GLOBAL IN for candidate filtering; time conversions via ToNanoSecs.
  • Validation msgs: Clearer errors for histogram/summary label requirements.
• API/Access control:
  • /api/v2/metrics/{metric_name}/metadata now requires EditAccess.
• Plumbing/Tests:
  • Module constructors updated to accept MetricsExplorer config; tests adjusted.

^{Written by Cursor Bugbot for commit ea5a620. This will update automatically on new commits. Configure here.}

[srikanthccv]

the multiple shards query modification is still pending. will let you know when it's bumped

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[therealpandey]

@nikhilmantri0902 we're reading config from the config package.

You can do:

metricsexplorer:
  threads:

OR

metricsexplorer:
  telemetrystore:
    threads:

(Deliberately not using the word clickhouse because the abstraction is called telemetrystore)

1. Create a config.go in pkg/modules/metricsexplorer.
2. Register this config in the config factory in signoz.
3. Inject this config in the module.

[nikhilmantri0902]

@therealpandey made changes for config via config factory.