Avoid mmap for 32-bit architecture

Author: gammazero

datastore.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"os"
 	"runtime"
 	"strings"
 	"sync"
@@ -107,8 +108,12 @@ func init() {
 	// reading (in some tests).
 	DefaultOptions.Options.ValueLogLoadingMode = options.FileIO
 
-	// Explicitly set this to mmap. This doesn't use much memory anyways.
-	DefaultOptions.Options.TableLoadingMode = options.MemoryMap
+	if os.Getenv("GOARCH") == "386" {
+		DefaultOptions.Options.TableLoadingMode = options.FileIO
+	} else {
+		// Explicitly set this to mmap. This doesn't use much memory anyways.
+		DefaultOptions.Options.TableLoadingMode = options.MemoryMap
+	}
 
 	// Reduce this from 64MiB to 16MiB. That means badger will hold on to
 	// 20MiB by default instead of 80MiB.

ds_test.go

@@ -5,12 +5,10 @@ import (
 	"context"
 	"crypto/rand"
 	"fmt"
-	"os"
 	"sort"
 	"testing"
 	"time"
 
-	options "github.com/dgraph-io/badger/options"
 	ds "github.com/ipfs/go-datastore"
 	dsq "github.com/ipfs/go-datastore/query"
 	dstest "github.com/ipfs/go-datastore/test"
@@ -634,11 +632,7 @@ func TestDiskUsage(t *testing.T) {
 	addTestCases(t, d, testcases)
 	d.Close()
 
-	dsOpts := DefaultOptions
-	if os.Getenv("GOARCH") == "386" {
-		dsOpts.Options.TableLoadingMode = options.FileIO
-	}
-	d, err = NewDatastore(path, &dsOpts)
+	d, err = NewDatastore(path, nil)
 	if err != nil {
 		t.Fatal(err)
 	}