The GitHub Actions job "Required Checks" on texera.git/main has failed.
Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).

Head commit for run:
056e39d0d9a46b25648e671cedee4337ffe3a29c / Matthew B. <[email protected]>
fix: Adding Floor to idle_time (#4900)

### What changes were proposed in this PR?
  
- idle_time was computed as total_execution_time - data_processing_time
- control_processing_time, with no lower bound — timing races could make
it negative and propagate bad values to the dashboard
- Added drift detection warning in update_total_execution_time when the
new total is less than the accumulated processing time
- Added monotonicity check in update_total_execution_time to catch
non-increasing timestamps
  - get_statistics() clamps idle_time to 0 via max(0, ...)  
 
### Any related issues, documentation, or discussions?

closes: #4892 

### How was this PR tested?
- Updated TestStatisticsManagerExecutionTime to assert idle_time == 0
for the overshoot case instead of -30
  - Removed the xfail marker that was tracking the unfixed bug

### Was this PR authored or co-authored using generative AI tooling?
<!--
If generative AI tooling has been used in the process of authoring this
PR,
please include the phrase: 'Generated-by: ' followed by the name of the
tool
and its version. If no, write 'No'. 
Please refer to the [ASF Generative Tooling
Guidance](https://www.apache.org/legal/generative-tooling.html) for
details.
-->
Co-Authored with Claude Opus 4.6 in Compliance with ASF

---------

Signed-off-by: Matthew B. <[email protected]>

Report URL: https://github.com/apache/texera/actions/runs/25350792517

With regards,
GitHub Actions via GitBox

Reply via email to