removed bleve-mem, as it duplicates mem kvStoreName

docs/api-ref.md

@@ -46,7 +46,6 @@ supported index types:
 * alias - supports fan-out of queries to multiple index targets
 * blackhole - ignores all incoming data and is not queryable; used for testing
 * bleve - full-text index powered by the bleve full-text-search engine
-* bleve-mem - full-text index powered by bleve (in memory only)
 
 **param: planParams**: optional, string (JSON), form parameter
 

pindex_impl_bleve.go

@@ -42,6 +42,9 @@ type BleveParams struct {
 func NewBleveParams() *BleveParams {
 	return &BleveParams{
 		Mapping: *bleve.NewIndexMapping(),
+		Store: map[string]interface{}{
+			"kvStoreName": bleve.Config.DefaultKVStore,
+		},
 	}
 }
 
@@ -113,19 +116,6 @@ func init() {
 			" powered by the bleve full-text-search engine",
 		StartSample: NewBleveParams(),
 	})
-
-	RegisterPIndexImplType("bleve-mem", &PIndexImplType{
-		Validate: ValidateBlevePIndexImpl,
-
-		New:   NewBlevePIndexImpl,
-		Open:  OpenBlevePIndexImpl,
-		Count: CountBlevePIndexImpl,
-		Query: QueryBlevePIndexImpl,
-
-		Description: "bleve-mem - full-text index" +
-			" powered by bleve (in memory only)",
-		StartSample: NewBleveParams(),
-	})
 }
 
 func ValidateBlevePIndexImpl(indexType, indexName, indexParams string) error {
@@ -146,20 +136,6 @@ func NewBlevePIndexImpl(indexType, indexParams, path string,
 		}
 	}
 
-	blevePath := path
-	if indexType == "bleve-mem" {
-		blevePath = "" // Force bleve to use memory-only storage.
-
-		// For a normal, non-empty path, bleve will create the
-		// directory (and also expects path not to exist yet
-		// beforehand).  And, for an empty path, we need to create the
-		// directory here because bleve won't do so.
-		err := os.MkdirAll(path, 0700)
-		if err != nil {
-			return nil, nil, err
-		}
-	}
-
 	kvStoreName, ok := bleveParams.Store["kvStoreName"].(string)
 	if !ok || kvStoreName == "" {
 		kvStoreName = bleve.Config.DefaultKVStore
@@ -184,7 +160,7 @@ func NewBlevePIndexImpl(indexType, indexParams, path string,
 	}
 
 	bindex, err :=
-		bleve.NewUsing(blevePath, &bleveParams.Mapping, kvStoreName, kvConfig)
+		bleve.NewUsing(path, &bleveParams.Mapping, kvStoreName, kvConfig)
 	if err != nil {
 		return nil, nil, fmt.Errorf("bleve: new index, path: %s,"+
 			" kvStoreName: %s, kvConfig: %#v, err: %s",
@@ -204,11 +180,6 @@ func NewBlevePIndexImpl(indexType, indexParams, path string,
 
 func OpenBlevePIndexImpl(indexType, path string,
 	restart func()) (PIndexImpl, Dest, error) {
-	if indexType == "bleve-mem" {
-		return nil, nil, fmt.Errorf("bleve: cannot re-open bleve-mem,"+
-			" path: %s", path)
-	}
-
 	buf, err := ioutil.ReadFile(path +
 		string(os.PathSeparator) + "PINDEX_BLEVE_META")
 	if err != nil {

pindex_test.go

@@ -165,27 +165,6 @@ func TestBlackholePIndexImpl(t *testing.T) {
 	}
 }
 
-func TestNewPIndexBleveMem(t *testing.T) {
-	emptyDir, _ := ioutil.TempDir("./tmp", "test")
-	defer os.RemoveAll(emptyDir)
-
-	pindex, err := NewPIndex(nil, "fake", "uuid",
-		"bleve-mem", "indexName", "indexUUID", "",
-		"sourceType", "sourceName", "sourceUUID", "sourceParams", "sourcePartitions",
-		PIndexPath(emptyDir, "fake"))
-	if pindex == nil || err != nil {
-		t.Errorf("expected NewPIndex to work")
-	}
-	pindex2, err := OpenPIndex(nil, PIndexPath(emptyDir, "fake"))
-	if pindex2 != nil || err == nil {
-		t.Errorf("expected OpenPIndex to fail for bleve-mem")
-	}
-	err = pindex.Close(true)
-	if err != nil {
-		t.Errorf("expected Close to work")
-	}
-}
-
 func TestErrorConsistencyWait(t *testing.T) {
 	e := &ErrorConsistencyWait{}
 	if e.Error() == "" {

rest_test.go

@@ -411,10 +411,6 @@ func TestHandlersForOneBleveIndexWithNILFeed(t *testing.T) {
 	testHandlersForOneBleveTypeIndexWithNILFeed(t, "bleve")
 }
 
-func TestHandlersForOneBleveMemIndexWithNILFeed(t *testing.T) {
-	testHandlersForOneBleveTypeIndexWithNILFeed(t, "bleve-mem")
-}
-
 func testHandlersForOneBleveTypeIndexWithNILFeed(t *testing.T, indexType string) {
 	emptyDir, _ := ioutil.TempDir("./tmp", "test")
 	defer os.RemoveAll(emptyDir)