Optimize $percentile rank computation by dropping representAsChecked in computeDiscreteRank

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 9.1.0-rc0, 9.1.0-rc1022
    • Affects Version/s: None
    • Component/s: None
    • None
    • Query Integration
    • Fully Compatible
    • QI 2025-02-17
    • 200
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Replaces the per-call `representAsChecked<int>()` optional/`tassert` round-trip in `computeDiscreteRank()` — the rank helper shared by `DiscretePercentile` and `TDigest`. This is run once per requested `$percentile` — with an `isfinite`-guarded `static_cast` (behavior identical for every input `parseP()` admits). sys-perf A/B measured `percentiles_expr` +20–30% on master (both classic and SBE execution engines) and `array_10_percentile_many100` −19% → +19% vs 8.2 on v8.3.

            Assignee:
            Finley Lau
            Reporter:
            Finley Lau
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: