Skip to content

Conversation

@han-yan01
Copy link
Contributor

Summary:
finishCpu += operator.getFinishCpu().roundTo(NANOSECONDS);

getFinishCpu has underlying issue that causes finisheCpu to overflow thus resulting in a negative result. This causes exception while reporting query completion event leading to stats not being reported. Fix it by setting finishCpu to max value when it overflows

Release Note

== NO RELEASE NOTE ==

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.
  • If adding new dependencies, verified they have an OpenSSF Scorecard score of 5.0 or higher (or obtained explicit TSC approval for lower scores).

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ... 
* ... 

Hive Connector Changes
* ... 
* ... 

If release note is NOT required, use:

== NO RELEASE NOTE ==

Summary:
finishCpu += operator.getFinishCpu().roundTo(NANOSECONDS);

getFinishCpu has underlying issue that causes finisheCpu to overflow
thus resulting in a negative result. This causes exception while
reporting query completion event leading to stats not being reported.
Fix it by setting finishCpu to max value when it overflows


# Release Note
```
== NO RELEASE NOTE ==
```
@han-yan01 han-yan01 requested a review from a team as a code owner November 12, 2025 19:33
@prestodb-ci prestodb-ci added the from:Meta PR from Meta label Nov 12, 2025
@stevechuck stevechuck assigned stevechuck and unassigned stevechuck Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:Meta PR from Meta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants