New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MOD-6002: Fixes for FT.PROFILE
's time reporting
#4264
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #4264 +/- ##
==========================================
+ Coverage 83.81% 84.35% +0.53%
==========================================
Files 192 192
Lines 33791 33460 -331
==========================================
- Hits 28323 28225 -98
+ Misses 5468 5235 -233 ☔ View full report in Codecov by Sentry. |
1b2b3e1
to
8bef3cb
Compare
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin 2.8
git worktree add -d .worktree/backport-4264-to-2.8 origin/2.8
cd .worktree/backport-4264-to-2.8
git switch --create backport-4264-to-2.8
git cherry-pick -x ea69686de07ee8fac53b8b253f4e570e6e458b22 6f5f24be9f51be3059363049dcc6c18d15dd18a5 8bef3cb191281aab05714dcdd097080ea6ddd6f5 bc58460d77d449db3dffd9843355525e93b6402e 4c9e20e527e5fcfee12be943c13452d35dd51588 e7ef2c6d2be7d2a01b6ef9cec030a35e8840300c b2b1462d335482d387a338fc036b08118b534e28 |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin 2.6
git worktree add -d .worktree/backport-4264-to-2.6 origin/2.6
cd .worktree/backport-4264-to-2.6
git switch --create backport-4264-to-2.6
git cherry-pick -x ea69686de07ee8fac53b8b253f4e570e6e458b22 6f5f24be9f51be3059363049dcc6c18d15dd18a5 8bef3cb191281aab05714dcdd097080ea6ddd6f5 bc58460d77d449db3dffd9843355525e93b6402e 4c9e20e527e5fcfee12be943c13452d35dd51588 e7ef2c6d2be7d2a01b6ef9cec030a35e8840300c b2b1462d335482d387a338fc036b08118b534e28 |
* revert transition to hires clock * fix * fix flaky test * Add test * Skip on sanitizer, reduce number of docs * Test coordinator timer, remove duplicate code * Add docs to test (remove flakiness)
* revert transition to hires clock * fix * fix flaky test * Add test * Skip on sanitizer, reduce number of docs * Test coordinator timer, remove duplicate code * Add docs to test (remove flakiness)
* revert transition to hires clock * fix * fix flaky test * Add test * Skip on sanitizer, reduce number of docs * Test coordinator timer, remove duplicate code * Add docs to test (remove flakiness)
Currently, our
FT.PROFILE
response is populated with many0
's, due to bad casting which cuts off the accumulated time per result in the aggregation pipeline. This PR reverts the changes introduced in #2989, back to working with theclock
function (which is still in use in other parts of the project).If we wish to support higher clock resolution later on, we should use the
struct timespec
that supports nano-second resolution rather than the clock's micro-seconds.Mark if applicable