Skip to content

Commit d8dc030

Browse files
committed
drop category
1 parent a8b5bd3 commit d8dc030

File tree

4 files changed

+11
-9
lines changed

4 files changed

+11
-9
lines changed

balancer/xds/edsbalancer/edsbalancer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ func (xdsB *EDSBalancer) updateDrops(dropPolicies []*edspb.ClusterLoadAssignment
138138
case percentpb.FractionalPercent_MILLION:
139139
denominator = 1000000
140140
}
141-
newDrops = append(newDrops, newDropper(numerator, denominator))
141+
newDrops = append(newDrops, newDropper(numerator, denominator, dropPolicy.GetCategory()))
142142

143143
// The following reading xdsB.drops doesn't need mutex because it can only
144144
// be updated by the code following.

balancer/xds/edsbalancer/edsbalancer_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -480,22 +480,22 @@ func TestDropPicker(t *testing.T) {
480480
{
481481
name: "one drop",
482482
drops: []*dropper{
483-
newDropper(1, 2),
483+
newDropper(1, 2, ""),
484484
},
485485
},
486486
{
487487
name: "two drops",
488488
drops: []*dropper{
489-
newDropper(1, 3),
490-
newDropper(1, 2),
489+
newDropper(1, 3, ""),
490+
newDropper(1, 2, ""),
491491
},
492492
},
493493
{
494494
name: "three drops",
495495
drops: []*dropper{
496-
newDropper(1, 3),
497-
newDropper(1, 4),
498-
newDropper(1, 2),
496+
newDropper(1, 3, ""),
497+
newDropper(1, 4, ""),
498+
newDropper(1, 2, ""),
499499
},
500500
},
501501
}

balancer/xds/edsbalancer/util.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,10 @@ type dropper struct {
2525
numerator uint32
2626
denominator uint32
2727
w wrr.WRR
28+
category string
2829
}
2930

30-
func newDropper(numerator, denominator uint32) *dropper {
31+
func newDropper(numerator, denominator uint32, category string) *dropper {
3132
w := newRandomWRR()
3233
w.Add(true, int64(numerator))
3334
w.Add(false, int64(denominator-numerator))
@@ -36,6 +37,7 @@ func newDropper(numerator, denominator uint32) *dropper {
3637
numerator: numerator,
3738
denominator: denominator,
3839
w: w,
40+
category: category,
3941
}
4042
}
4143

balancer/xds/edsbalancer/util_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ func TestDropper(t *testing.T) {
106106
}
107107
for _, tt := range tests {
108108
t.Run(tt.name, func(t *testing.T) {
109-
d := newDropper(tt.args.numerator, tt.args.denominator)
109+
d := newDropper(tt.args.numerator, tt.args.denominator, "")
110110
var (
111111
dCount int
112112
wantCount = int(tt.args.numerator) * repeat

0 commit comments

Comments
 (0)