apache / apache/arrow

[C++] Optimize Decimal128 abs hotspot; add exec‑only TPC‑H Q1 benchmark

Open
#48,931 0 comments 0 reactions 1 assignee Claimed by @Manas103 View on GitHub
Component: C++ Type: enhancement
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 13h
Merged PRs (30d)
88

Description

### Describe the enhancement requested

While profiling `BM_Tpch_Q1_ExecOnly in Acero`, a significant CPU hotspot appears in `BasicDecimal128::Abs()` (via decimal math in Q1). This PR proposes a small optimization to avoid temporary creation/operator< in `Abs()` and adds an exec‑only TPC‑H Q1 benchmark that isolates execution from data generation to make profiling/benchmarking more representative.

This enhancement should improve performance in decimal‑heavy compute and make it easier to identify execution hotspots in Acero.

### Component(s)

C++

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.