Skip to content

Commit 9a6ff71

Browse files
committed
kv: timeutil.Now -> crtime.NowMono conversions
Epic: none Release note: None
1 parent b27a426 commit 9a6ff71

File tree

6 files changed

+17
-16
lines changed

6 files changed

+17
-16
lines changed

pkg/kv/kvclient/kvcoord/dist_sender_mux_rangefeed.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import (
2525
"github.com/cockroachdb/cockroach/pkg/util/retry"
2626
"github.com/cockroachdb/cockroach/pkg/util/syncutil"
2727
"github.com/cockroachdb/cockroach/pkg/util/timeutil"
28+
"github.com/cockroachdb/crlib/crtime"
2829
"github.com/cockroachdb/errors"
2930
"github.com/cockroachdb/logtags"
3031
)
@@ -67,9 +68,9 @@ func muxRangeFeed(
6768
) (retErr error) {
6869
if log.V(1) {
6970
log.KvExec.Infof(ctx, "Establishing MuxRangeFeed (%s...; %d spans)", spans[0], len(spans))
70-
start := timeutil.Now()
71+
start := crtime.NowMono()
7172
defer func() {
72-
log.KvExec.Infof(ctx, "MuxRangeFeed terminating after %s with err=%v", timeutil.Since(start), retErr)
73+
log.KvExec.Infof(ctx, "MuxRangeFeed terminating after %s with err=%v", start.Elapsed(), retErr)
7374
}()
7475
}
7576

@@ -382,10 +383,10 @@ func (m *rangefeedMuxer) startNodeMuxRangeFeed(
382383

383384
if log.V(1) {
384385
log.KvExec.Infof(ctx, "Establishing MuxRangeFeed to node %d", nodeID)
385-
start := timeutil.Now()
386+
start := crtime.NowMono()
386387
defer func() {
387388
log.KvExec.Infof(ctx, "MuxRangeFeed to node %d terminating after %s with err=%v",
388-
nodeID, timeutil.Since(start), retErr)
389+
nodeID, start.Elapsed(), retErr)
389390
}()
390391
}
391392

pkg/kv/kvclient/rangefeed/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ go_library(
2929
"//pkg/util/retry",
3030
"//pkg/util/span",
3131
"//pkg/util/stop",
32-
"//pkg/util/timeutil",
32+
"@com_github_cockroachdb_crlib//crtime",
3333
"@com_github_cockroachdb_errors//:errors",
3434
"@com_github_cockroachdb_logtags//:logtags",
3535
"@com_github_cockroachdb_redact//:redact",

pkg/kv/kvclient/rangefeed/rangefeed.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import (
2626
"github.com/cockroachdb/cockroach/pkg/util/retry"
2727
"github.com/cockroachdb/cockroach/pkg/util/span"
2828
"github.com/cockroachdb/cockroach/pkg/util/stop"
29-
"github.com/cockroachdb/cockroach/pkg/util/timeutil"
29+
"github.com/cockroachdb/crlib/crtime"
3030
"github.com/cockroachdb/errors"
3131
"github.com/cockroachdb/logtags"
3232
"github.com/cockroachdb/redact"
@@ -361,7 +361,7 @@ func (f *RangeFeed) run(ctx context.Context, frontier span.Frontier, resumeWithF
361361
log.Eventf(ctx, "starting rangefeed from %v on %v", ts, f.spansDebugStr)
362362
}
363363

364-
start := timeutil.Now()
364+
start := crtime.NowMono()
365365

366366
rangeFeedTask := func(ctx context.Context) error {
367367
if f.invoker == nil {
@@ -410,7 +410,7 @@ func (f *RangeFeed) run(ctx context.Context, frontier span.Frontier, resumeWithF
410410
return
411411
}
412412

413-
ranFor := timeutil.Since(start)
413+
ranFor := start.Elapsed()
414414
log.VEventf(ctx, 1, "restarting rangefeed for %v after %v",
415415
f.spansDebugStr, ranFor)
416416
if f.knobs != nil && f.knobs.OnRangefeedRestart != nil {

pkg/kv/kvserver/rangefeed/buffered_registration.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import (
1616
"github.com/cockroachdb/cockroach/pkg/util/log"
1717
"github.com/cockroachdb/cockroach/pkg/util/retry"
1818
"github.com/cockroachdb/cockroach/pkg/util/syncutil"
19-
"github.com/cockroachdb/cockroach/pkg/util/timeutil"
19+
"github.com/cockroachdb/crlib/crtime"
2020
"github.com/cockroachdb/errors"
2121
)
2222

@@ -304,10 +304,10 @@ func (br *bufferedRegistration) maybeRunCatchUpScan(ctx context.Context) error {
304304
if catchUpIter == nil {
305305
return nil
306306
}
307-
start := timeutil.Now()
307+
start := crtime.NowMono()
308308
defer func() {
309309
catchUpIter.Close()
310-
br.metrics.RangeFeedCatchUpScanNanos.Inc(timeutil.Since(start).Nanoseconds())
310+
br.metrics.RangeFeedCatchUpScanNanos.Inc(start.Elapsed().Nanoseconds())
311311
}()
312312

313313
return catchUpIter.CatchUpScan(ctx, br.stream.SendUnbuffered, br.withDiff, br.withFiltering, br.withOmitRemote, br.bulkDelivery)

pkg/kv/kvserver/rangefeed/unbuffered_registration.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import (
1414
"github.com/cockroachdb/cockroach/pkg/util/hlc"
1515
"github.com/cockroachdb/cockroach/pkg/util/retry"
1616
"github.com/cockroachdb/cockroach/pkg/util/syncutil"
17-
"github.com/cockroachdb/cockroach/pkg/util/timeutil"
1817
"github.com/cockroachdb/crlib/crtime"
1918
"github.com/cockroachdb/errors"
2019
)
@@ -359,10 +358,10 @@ func (ubr *unbufferedRegistration) maybeRunCatchUpScan(ctx context.Context) erro
359358
if catchUpIter == nil {
360359
return nil
361360
}
362-
start := timeutil.Now()
361+
start := crtime.NowMono()
363362
defer func() {
364363
catchUpIter.Close()
365-
ubr.metrics.RangeFeedCatchUpScanNanos.Inc(timeutil.Since(start).Nanoseconds())
364+
ubr.metrics.RangeFeedCatchUpScanNanos.Inc(start.Elapsed().Nanoseconds())
366365
}()
367366

368367
return catchUpIter.CatchUpScan(ctx, ubr.stream.SendUnbuffered, ubr.withDiff, ubr.withFiltering,

pkg/kv/kvserver/replica_rangefeed.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ import (
3333
"github.com/cockroachdb/cockroach/pkg/util/syncutil/singleflight"
3434
"github.com/cockroachdb/cockroach/pkg/util/timeutil"
3535
"github.com/cockroachdb/cockroach/pkg/util/uuid"
36+
"github.com/cockroachdb/crlib/crtime"
3637
"github.com/cockroachdb/errors"
3738
)
3839

@@ -414,9 +415,9 @@ func (r *Replica) updateRangefeedFilterLocked() bool {
414415
// to believe that is blocks the raftMu in practice.
415416
func logSlowRangefeedRegistration(ctx context.Context) func() {
416417
const slowRaftMuWarnThreshold = 20 * time.Millisecond
417-
start := timeutil.Now()
418+
start := crtime.NowMono()
418419
return func() {
419-
elapsed := timeutil.Since(start)
420+
elapsed := start.Elapsed()
420421
if elapsed >= slowRaftMuWarnThreshold {
421422
log.KvDistribution.Warningf(ctx, "rangefeed registration took %s", elapsed)
422423
}

0 commit comments

Comments
 (0)