Skip to content

Commit

Permalink
LongAdder note
Browse files Browse the repository at this point in the history
  • Loading branch information
sergiitk committed Nov 12, 2024
1 parent e56c032 commit 29f6fe7
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions xds/src/main/java/io/grpc/xds/internal/rlqs/RlqsBucket.java
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ public class RlqsBucket {

// TODO(sergiitk): [impl] consider AtomicLongFieldUpdater
private final AtomicLong lastSnapshotTimeNanos = new AtomicLong(-1);

Check warning on line 36 in xds/src/main/java/io/grpc/xds/internal/rlqs/RlqsBucket.java

View check run for this annotation

Codecov / codecov/patch

xds/src/main/java/io/grpc/xds/internal/rlqs/RlqsBucket.java#L36

Added line #L36 was not covered by tests

// TODO(sergiitk): [impl] consider java.util.concurrent.atomic.LongAdder for counters
private final AtomicLong numRequestsAllowed = new AtomicLong();
private final AtomicLong numRequestsDenied = new AtomicLong();

Check warning on line 40 in xds/src/main/java/io/grpc/xds/internal/rlqs/RlqsBucket.java

View check run for this annotation

Codecov / codecov/patch

xds/src/main/java/io/grpc/xds/internal/rlqs/RlqsBucket.java#L39-L40

Added lines #L39 - L40 were not covered by tests

Expand Down

0 comments on commit 29f6fe7

Please sign in to comment.