#19705
89%

Ran 20 Feb 2025 04:35AM UTC

Jobs 1

Files 609

Run time 1min

Badge

Embed ▾

Committed 20 Feb 2025 04:25AM UTC coverage: 88.616% (+0.03%) from 88.589%

Build # #19705

Build Type

push

github

Committed by

web-flow

Commit Message

xds: explicitly set request hash key for the ring hash LB policy

Implements [gRFC A76: explicitly setting the request hash key for the
ring hash LB policy][A76]
* Explictly setting the request hash key is guarded by the
  `GRPC_EXPERIMENTAL_RING_HASH_SET_REQUEST_HASH_KEY` environment
  variable until API stabilized. 

Tested:
* Verified end-to-end by spinning up multiple gRPC servers and a gRPC
  client that injects a custom service (load balancing) config with
  `ring_hash_experimental` and a custom `request_hash_header` (with
  NO associated value in the metadata headers) which generates a random
  hash for each request to the ring hash LB. Verified picks/RPCs are
  split evenly/uniformly across all backends.
* Ran affected unit tests with thread sanitizer and 1000 iterations to
  prevent data races.

[A76]: https://github.com/grpc/proposal/blob/master/A76-ring-hash-improvements.md#explicitly-setting-the-request-hash-key

Run Details

34305 of 38712 relevant lines covered (88.62%)

0.89 hits per line

Uncovered Existing Lines

Lines	Coverage	∆	File
1	87.88	2.69%	../xds/src/main/java/io/grpc/xds/RingHashLoadBalancerProvider.java
2	81.05	-2.11%	../okhttp/src/main/java/io/grpc/okhttp/ExceptionHandlingFrameWriter.java
3	95.07	-0.44%	../core/src/main/java/io/grpc/internal/RetriableStream.java
12	94.8	2.93%	../xds/src/main/java/io/grpc/xds/RingHashLoadBalancer.java

Jobs

ID	Job ID	Ran	Files	Coverage
1	#19705.1	20 Feb 2025 04:35AM UTC	609	88.62

grpc / grpc-java / #19705
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #19705

grpc / grpc-java / #19705 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build #19705

grpc / grpc-java / #19705
89%

README BADGES
x