Skip to content

[2.6] [MOD-10622] Change time measurement method in FT.PROFILE#6659

Merged
lerman25 merged 4 commits into2.6from
backport-6574-to-2.6
Aug 17, 2025
Merged

[2.6] [MOD-10622] Change time measurement method in FT.PROFILE#6659
lerman25 merged 4 commits into2.6from
backport-6574-to-2.6

Conversation

@lerman25
Copy link
Copy Markdown
Collaborator

@lerman25 lerman25 commented Aug 14, 2025

backport #6574 to 2.6
This PR also fixed a typo in ft.profile response
Post Proccessing Time -> Post Processing Time

@lerman25 lerman25 changed the base branch from master to 2.6 August 14, 2025 15:02
@lerman25 lerman25 marked this pull request as draft August 14, 2025 15:04
@lerman25 lerman25 marked this pull request as ready for review August 14, 2025 15:04
GuyAv46
GuyAv46 previously approved these changes Aug 14, 2025
View reviewed changes
GuyAv46
GuyAv46 previously approved these changes Aug 14, 2025
View reviewed changes
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 14, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 98.48485% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.04%. Comparing base (24db9e8) to head (4153e24).

⚠️ Current head 4153e24 differs from pull request most recent head dcd209f

Please upload reports for the commit dcd209f to get more accurate results.

Files with missing lines Patch % Lines
src/index.c 94.73% 1 Missing ⚠️

❌ Your project check has failed because the head coverage (80.28%) is below the adjusted base coverage (85.46%). You can increase the head coverage or adjust the Removed Code Behavior.

❗ There is a different number of reports uploaded between BASE (24db9e8) and HEAD (4153e24). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (24db9e8) HEAD (4153e24)
unit 1 0
Additional details and impacted files 
@@            Coverage Diff             @@
##              2.6    #6659      +/-   ##
==========================================
- Coverage   86.40%   80.04%   -6.37%     
==========================================
  Files         185      186       +1     
  Lines       32253    32269      +16     
==========================================
- Hits        27869    25830    -2039     
- Misses       4384     6439    +2055
Flag Coverage Δ
flow 80.04% <98.48%> (-0.28%) ⬇️
unit ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lerman25 lerman25 added this pull request to the merge queue Aug 17, 2025
Merged via the queue into 2.6 with commit 8abc93a Aug 17, 2025
10 checks passed
@lerman25 lerman25 deleted the backport-6574-to-2.6 branch August 17, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GuyAv46 GuyAv46 GuyAv46 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lerman25 @GuyAv46