[I] Improve performance of update metrics [datafusion-comet]

[via GitHub]

wForget opened a new issue, #1328:
URL: https://github.com/apache/datafusion-comet/issues/1328

   ### What is the problem the feature request solves?
   
   When I profiled tpcds q64, I found that update_metrics took 8.9 % of task 
time.
   
   
![Image](https://github.com/user-attachments/assets/970c3313-9522-42f9-9d82-61d0e480c3fc)
   
   ### Describe the potential solution
   
   I propose the following improvements to optimize update metrics:
   + Define a `NativeMetricNode` proto type to pass all metric nodes at once to 
avoid recursive jni calls.
   + Call update metrics when releasing plan to reduce the number of calls.
   
   ### Additional context
   
   _No response_


-- 
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: github-unsubscr...@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org