scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-09-13 08:01:02 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

change test case to reflect cache loading before test

Browse source
This commit is contained in:
Samir Patel 2021-05-28 09:28:35 -05:00 • committed by Samir Patel
parent fe0bb20658
commit eff3b25b97
2 changed files with 27 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
server/handler_test.go
View file

@ -532,7 +532,7 @@ func TestHandler_Endpoints(t *testing.T) {
for _, nodeUsage := range nodeUsages {
t.Logf("Len of Indexes: %+v\n", len(nodeUsage.Disk.IndexUsage))
numIndexes := len(nodeUsage.Disk.IndexUsage)
// numIndexes := len(nodeUsage.Disk.IndexUsage)
fmt.Printf("Node Usage: %+v\n", nodeUsage)
fmt.Printf("Disk Usage: %+v\n", nodeUsage.Disk)
for k, v := range nodeUsage.Disk.IndexUsage {
@ -540,18 +540,36 @@ func TestHandler_Endpoints(t *testing.T) {
}
// if nodeUsage.Disk.TotalUse < 75000 || nodeUsage.Disk.TotalUse > 700000 {
if nodeUsage.Disk.TotalUse < 1 {
// Usage measurements are not consistent between machines, or
// over time, as features and implementations change, so checking
// for a range of sizes may be most useful way to test the details of this.
t.Fatalf("expected some disk use, got %d", nodeUsage.Disk.TotalUse)
}
if nodeUsage.Disk.Capacity < 1 {
// Usage measurements are not consistent between machines, or
// over time, as features and implementations change, so checking
// for a range of sizes may be most useful way to test the details of this.
t.Fatalf("expected 75k < total < 500k, got %d", nodeUsage.Disk.TotalUse)
}
if numIndexes != 3 {
t.Fatalf("wrong length index usage list: expected %d, got %d", 2, numIndexes)
if nodeUsage.Memory.TotalUse < 1 {
// Usage measurements are not consistent between machines, or
// over time, as features and implementations change, so checking
// for a range of sizes may be most useful way to test the details of this.
t.Fatalf("expected 75k < total < 500k, got %d", nodeUsage.Disk.TotalUse)
}
numFields := len(nodeUsage.Disk.IndexUsage["i1"].Fields)
if numFields != len(i1.Fields()) {
t.Fatalf("wrong length field usage list: expected %d, got %d", len(i1.Fields()), numFields)
if nodeUsage.Memory.Capacity < 1 {
// Usage measurements are not consistent between machines, or
// over time, as features and implementations change, so checking
// for a range of sizes may be most useful way to test the details of this.
t.Fatalf("expected 75k < total < 500k, got %d", nodeUsage.Disk.TotalUse)
}
// if numIndexes != 3 {
// t.Fatalf("wrong length index usage list: expected %d, got %d", 2, numIndexes)
// }
// numFields := len(nodeUsage.Disk.IndexUsage["i1"].Fields)
// if numFields != len(i1.Fields()) {
// t.Fatalf("wrong length field usage list: expected %d, got %d", len(i1.Fields()), numFields)
// }
}
})

7
txfactory.go
View file

@ -26,7 +26,6 @@ import (
"sync"
"syscall"
"text/tabwriter"
"time"
"github.com/pilosa/pilosa/v2/hash"
"github.com/pilosa/pilosa/v2/roaring"
@ -576,9 +575,9 @@ func (f *TxFactory) DumpAll() {
func (f *TxFactory) IndexUsageDetails() (map[string]IndexUsage, uint64, error) {
indexUsage := make(map[string]IndexUsage)
for len(f.holder.Indexes()) == 0 {
time.Sleep(time.Second)
}
// for len(f.holder.Indexes()) == 0 {
// time.Sleep(time.Second)
// }
fmt.Printf("f.holder: %+v\n", f.holder)
holderPath, err := expandDirName(f.holder.path)