malinjawi opened a new pull request, #13358:
URL: https://github.com/apache/gluten/pull/13358

   ## What changes are proposed in this pull request?
   
   Part of #9003. This is the JVM half of #13165, split out to make review 
easier.
   
   - `IncrementMetricCall` (gluten-substrait): an unevaluable pass-through 
expression that is sent to the backend as a call to one of its counter 
functions.
   - `BackendSettingsApi.supportNativeIncrementMetric()`, false by default. The 
Velox backend turns it on in #13165.
   - The Velox metrics path reads the counters' processed-row counts 
(`expressionStats`) from the native operator metrics, merges them across 
operators, and `ProjectMetricsUpdater` credits each SQL metric from its counter.
   
   Nothing emits `IncrementMetricCall` yet, so behavior doesn't change. The 
native functions are in #13357.
   
   ## How was this patch tested?
   
   New `ProjectMetricsUpdaterSuite`: a counter is credited with the rows its 
function processed, a counter that never ran with zero, and the existing 
`increment_metric` path still with the output rows.
   
   ## Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: IBM Bob
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to