mirror of
https://github.com/etcd-io/etcd.git
synced 2024-09-27 06:25:44 +00:00

Grow the inflights buffer as needed instead of preallocating it to its max size. This avoids preallocating a lot of unnecessary space (8*MaxInflightMsgs) when using lots of raft groups while still allowing for a reasonable MaxInflightMsgs configuration.
190 lines
3.6 KiB
Go
190 lines
3.6 KiB
Go
// Copyright 2015 The etcd Authors
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package raft
|
|
|
|
import (
|
|
"reflect"
|
|
"testing"
|
|
)
|
|
|
|
func TestInflightsAdd(t *testing.T) {
|
|
// no rotating case
|
|
in := &inflights{
|
|
size: 10,
|
|
buffer: make([]uint64, 10),
|
|
}
|
|
|
|
for i := 0; i < 5; i++ {
|
|
in.add(uint64(i))
|
|
}
|
|
|
|
wantIn := &inflights{
|
|
start: 0,
|
|
count: 5,
|
|
size: 10,
|
|
// ↓------------
|
|
buffer: []uint64{0, 1, 2, 3, 4, 0, 0, 0, 0, 0},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn)
|
|
}
|
|
|
|
for i := 5; i < 10; i++ {
|
|
in.add(uint64(i))
|
|
}
|
|
|
|
wantIn2 := &inflights{
|
|
start: 0,
|
|
count: 10,
|
|
size: 10,
|
|
// ↓---------------------------
|
|
buffer: []uint64{0, 1, 2, 3, 4, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn2) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn2)
|
|
}
|
|
|
|
// rotating case
|
|
in2 := &inflights{
|
|
start: 5,
|
|
size: 10,
|
|
buffer: make([]uint64, 10),
|
|
}
|
|
|
|
for i := 0; i < 5; i++ {
|
|
in2.add(uint64(i))
|
|
}
|
|
|
|
wantIn21 := &inflights{
|
|
start: 5,
|
|
count: 5,
|
|
size: 10,
|
|
// ↓------------
|
|
buffer: []uint64{0, 0, 0, 0, 0, 0, 1, 2, 3, 4},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in2, wantIn21) {
|
|
t.Fatalf("in = %+v, want %+v", in2, wantIn21)
|
|
}
|
|
|
|
for i := 5; i < 10; i++ {
|
|
in2.add(uint64(i))
|
|
}
|
|
|
|
wantIn22 := &inflights{
|
|
start: 5,
|
|
count: 10,
|
|
size: 10,
|
|
// -------------- ↓------------
|
|
buffer: []uint64{5, 6, 7, 8, 9, 0, 1, 2, 3, 4},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in2, wantIn22) {
|
|
t.Fatalf("in = %+v, want %+v", in2, wantIn22)
|
|
}
|
|
}
|
|
|
|
func TestInflightFreeTo(t *testing.T) {
|
|
// no rotating case
|
|
in := newInflights(10)
|
|
for i := 0; i < 10; i++ {
|
|
in.add(uint64(i))
|
|
}
|
|
|
|
in.freeTo(4)
|
|
|
|
wantIn := &inflights{
|
|
start: 5,
|
|
count: 5,
|
|
size: 10,
|
|
// ↓------------
|
|
buffer: []uint64{0, 1, 2, 3, 4, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn)
|
|
}
|
|
|
|
in.freeTo(8)
|
|
|
|
wantIn2 := &inflights{
|
|
start: 9,
|
|
count: 1,
|
|
size: 10,
|
|
// ↓
|
|
buffer: []uint64{0, 1, 2, 3, 4, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn2) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn2)
|
|
}
|
|
|
|
// rotating case
|
|
for i := 10; i < 15; i++ {
|
|
in.add(uint64(i))
|
|
}
|
|
|
|
in.freeTo(12)
|
|
|
|
wantIn3 := &inflights{
|
|
start: 3,
|
|
count: 2,
|
|
size: 10,
|
|
// ↓-----
|
|
buffer: []uint64{10, 11, 12, 13, 14, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn3) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn3)
|
|
}
|
|
|
|
in.freeTo(14)
|
|
|
|
wantIn4 := &inflights{
|
|
start: 0,
|
|
count: 0,
|
|
size: 10,
|
|
// ↓
|
|
buffer: []uint64{10, 11, 12, 13, 14, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn4) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn4)
|
|
}
|
|
}
|
|
|
|
func TestInflightFreeFirstOne(t *testing.T) {
|
|
in := newInflights(10)
|
|
for i := 0; i < 10; i++ {
|
|
in.add(uint64(i))
|
|
}
|
|
|
|
in.freeFirstOne()
|
|
|
|
wantIn := &inflights{
|
|
start: 1,
|
|
count: 9,
|
|
size: 10,
|
|
// ↓------------------------
|
|
buffer: []uint64{0, 1, 2, 3, 4, 5, 6, 7, 8, 9},
|
|
}
|
|
|
|
if !reflect.DeepEqual(in, wantIn) {
|
|
t.Fatalf("in = %+v, want %+v", in, wantIn)
|
|
}
|
|
}
|