123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384 |
- package sub_coordinator
- import (
- "github.com/seaweedfs/seaweedfs/weed/mq/pub_balancer"
- "reflect"
- "testing"
- )
- func Test_doBalanceSticky(t *testing.T) {
- type args struct {
- partitions []*pub_balancer.PartitionSlotToBroker
- consumerInstanceIds []*ConsumerGroupInstance
- prevMapping *PartitionSlotToConsumerInstanceList
- }
- tests := []struct {
- name string
- args args
- wantPartitionSlots []*PartitionSlotToConsumerInstance
- }{
- {
- name: "1 consumer instance, 1 partition",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: nil,
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-1",
- },
- },
- },
- {
- name: "2 consumer instances, 1 partition",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: nil,
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-1",
- },
- },
- },
- {
- name: "1 consumer instance, 2 partitions",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: nil,
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-1",
- },
- },
- },
- {
- name: "2 consumer instances, 2 partitions",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: nil,
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- {
- name: "2 consumer instances, 2 partitions, 1 deleted consumer instance",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: &PartitionSlotToConsumerInstanceList{
- PartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-3",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- {
- name: "2 consumer instances, 2 partitions, 1 new consumer instance",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-3",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: &PartitionSlotToConsumerInstanceList{
- PartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-3",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-3",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- {
- name: "2 consumer instances, 2 partitions, 1 new partition",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- {
- RangeStart: 100,
- RangeStop: 150,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: &PartitionSlotToConsumerInstanceList{
- PartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- {
- RangeStart: 100,
- RangeStop: 150,
- AssignedInstanceId: "consumer-instance-1",
- },
- },
- },
- {
- name: "2 consumer instances, 2 partitions, 1 new partition, 1 new consumer instance",
- args: args{
- partitions: []*pub_balancer.PartitionSlotToBroker{
- {
- RangeStart: 0,
- RangeStop: 50,
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- },
- {
- RangeStart: 100,
- RangeStop: 150,
- },
- },
- consumerInstanceIds: []*ConsumerGroupInstance{
- {
- InstanceId: "consumer-instance-1",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-2",
- MaxPartitionCount: 1,
- },
- {
- InstanceId: "consumer-instance-3",
- MaxPartitionCount: 1,
- },
- },
- prevMapping: &PartitionSlotToConsumerInstanceList{
- PartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- },
- },
- },
- wantPartitionSlots: []*PartitionSlotToConsumerInstance{
- {
- RangeStart: 0,
- RangeStop: 50,
- AssignedInstanceId: "consumer-instance-1",
- },
- {
- RangeStart: 50,
- RangeStop: 100,
- AssignedInstanceId: "consumer-instance-2",
- },
- {
- RangeStart: 100,
- RangeStop: 150,
- AssignedInstanceId: "consumer-instance-3",
- },
- },
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- if gotPartitionSlots := doBalanceSticky(tt.args.partitions, tt.args.consumerInstanceIds, tt.args.prevMapping); !reflect.DeepEqual(gotPartitionSlots, tt.wantPartitionSlots) {
- t.Errorf("doBalanceSticky() = %v, want %v", gotPartitionSlots, tt.wantPartitionSlots)
- }
- })
- }
- }
|