fix: smartTraceAlgo for new schema

[nityanandagohain]

Part of #5713

This was not added to searchTracesV2, so added it now .

---

Important

Enhance SearchTracesV2 in reader.go to support a smart trace algorithm for large trace data handling.

• Functionality:
  • SearchTracesV2 in reader.go now accepts a smartTraceAlgorithm function parameter.
  • Applies smartTraceAlgorithm if searchScanResponses exceeds SpansRenderLimit and SmartTraceDetail feature is enabled.
  • Logs telemetry event TELEMETRY_EVENT_LARGE_TRACE_OPENED if smart algorithm is applied.
• Error Handling:
  • Returns error if smartTraceAlgorithm fails.
• Telemetry:
  • Sends telemetry data when trace size exceeds MaxSpansInTrace.

^{This description was created by for f6687c1. It will automatically update as commits are pushed.}

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to f6687c1 in 47 seconds

More details

• Looked at 57 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. pkg/query-service/app/clickhouseReader/reader.go:1320

• Draft comment:
  Do not add non-ClickHouse related functions like GetDisks to the ClickHouseReader interface. Use the DAO in the telemetry instance instead. This applies to other similar functions as well.
• Reason this comment was not posted:
  Comment was on unchanged code.

Workflow ID: wflow_Ty0DCjqq6waohson

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.