mirror of
https://gitee.com/milvus-io/milvus.git
synced 2025-12-07 01:28:27 +08:00
26d26aa8ab
Signed-off-by: cai.zhang <cai.zhang@zilliz.com> Signed-off-by: cai.zhang <cai.zhang@zilliz.com>
483 lines
14 KiB
Go
483 lines
14 KiB
Go
// Licensed to the LF AI & Data foundation under one
|
|
// or more contributor license agreements. See the NOTICE file
|
|
// distributed with this work for additional information
|
|
// regarding copyright ownership. The ASF licenses this file
|
|
// to you 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 proxy
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/milvus-io/milvus-proto/go-api/commonpb"
|
|
"github.com/milvus-io/milvus-proto/go-api/milvuspb"
|
|
"github.com/milvus-io/milvus-proto/go-api/schemapb"
|
|
"github.com/milvus-io/milvus/internal/proto/indexpb"
|
|
"github.com/milvus-io/milvus/internal/proto/querypb"
|
|
"github.com/milvus-io/milvus/internal/util/funcutil"
|
|
"github.com/milvus-io/milvus/internal/util/typeutil"
|
|
)
|
|
|
|
func TestGetIndexStateTask_Execute(t *testing.T) {
|
|
dbName := funcutil.GenRandomStr()
|
|
collectionName := funcutil.GenRandomStr()
|
|
collectionID := UniqueID(1)
|
|
fieldName := funcutil.GenRandomStr()
|
|
indexName := ""
|
|
ctx := context.Background()
|
|
|
|
rootCoord := newMockRootCoord()
|
|
indexCoord := newMockIndexCoord()
|
|
queryCoord := NewQueryCoordMock()
|
|
|
|
gist := &getIndexStateTask{
|
|
GetIndexStateRequest: &milvuspb.GetIndexStateRequest{
|
|
Base: &commonpb.MsgBase{},
|
|
DbName: dbName,
|
|
CollectionName: collectionName,
|
|
FieldName: fieldName,
|
|
IndexName: indexName,
|
|
},
|
|
ctx: ctx,
|
|
indexCoord: indexCoord,
|
|
rootCoord: rootCoord,
|
|
result: &milvuspb.GetIndexStateResponse{
|
|
Status: &commonpb.Status{ErrorCode: commonpb.ErrorCode_UnexpectedError, Reason: "mock"},
|
|
State: commonpb.IndexState_Unissued,
|
|
},
|
|
collectionID: collectionID,
|
|
}
|
|
|
|
shardMgr := newShardClientMgr()
|
|
// failed to get collection id.
|
|
_ = InitMetaCache(ctx, rootCoord, queryCoord, shardMgr)
|
|
assert.Error(t, gist.Execute(ctx))
|
|
|
|
rootCoord.DescribeCollectionFunc = func(ctx context.Context, request *milvuspb.DescribeCollectionRequest) (*milvuspb.DescribeCollectionResponse, error) {
|
|
return &milvuspb.DescribeCollectionResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
Schema: newTestSchema(),
|
|
CollectionID: collectionID,
|
|
CollectionName: request.CollectionName,
|
|
}, nil
|
|
}
|
|
|
|
indexCoord.GetIndexStateFunc = func(ctx context.Context, request *indexpb.GetIndexStateRequest) (*indexpb.GetIndexStateResponse, error) {
|
|
return &indexpb.GetIndexStateResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
State: commonpb.IndexState_Finished,
|
|
FailReason: "",
|
|
}, nil
|
|
}
|
|
|
|
assert.NoError(t, gist.Execute(ctx))
|
|
assert.Equal(t, commonpb.IndexState_Finished, gist.result.GetState())
|
|
}
|
|
|
|
func TestDropIndexTask_PreExecute(t *testing.T) {
|
|
collectionName := "collection1"
|
|
collectionID := UniqueID(1)
|
|
fieldName := "field1"
|
|
indexName := "_default_idx_101"
|
|
|
|
Params.Init()
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
CollectionIDs: nil,
|
|
}, nil
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
ic := newMockIndexCoord()
|
|
ctx := context.Background()
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
|
|
mockCache := newMockCache()
|
|
mockCache.setGetIDFunc(func(ctx context.Context, collectionName string) (typeutil.UniqueID, error) {
|
|
return collectionID, nil
|
|
})
|
|
globalMetaCache = mockCache
|
|
|
|
dit := dropIndexTask{
|
|
ctx: ctx,
|
|
DropIndexRequest: &milvuspb.DropIndexRequest{
|
|
Base: &commonpb.MsgBase{
|
|
MsgType: 0,
|
|
MsgID: 0,
|
|
Timestamp: 0,
|
|
SourceID: 0,
|
|
TargetID: 0,
|
|
},
|
|
CollectionName: collectionName,
|
|
FieldName: fieldName,
|
|
IndexName: indexName,
|
|
},
|
|
indexCoord: ic,
|
|
queryCoord: qc,
|
|
result: nil,
|
|
collectionID: collectionID,
|
|
}
|
|
|
|
t.Run("normal", func(t *testing.T) {
|
|
err := dit.PreExecute(ctx)
|
|
assert.NoError(t, err)
|
|
})
|
|
|
|
t.Run("get collectionID error", func(t *testing.T) {
|
|
mockCache := newMockCache()
|
|
mockCache.setGetIDFunc(func(ctx context.Context, collectionName string) (typeutil.UniqueID, error) {
|
|
return 0, errors.New("error")
|
|
})
|
|
globalMetaCache = mockCache
|
|
err := dit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
mockCache.setGetIDFunc(func(ctx context.Context, collectionName string) (typeutil.UniqueID, error) {
|
|
return collectionID, nil
|
|
})
|
|
globalMetaCache = mockCache
|
|
|
|
t.Run("coll has been loaded", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
CollectionIDs: []int64{collectionID},
|
|
}, nil
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
dit.queryCoord = qc
|
|
|
|
err := dit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("show collection error", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return nil, errors.New("error")
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
dit.queryCoord = qc
|
|
|
|
err := dit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("show collection fail", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_UnexpectedError,
|
|
Reason: "fail reason",
|
|
},
|
|
}, nil
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
dit.queryCoord = qc
|
|
|
|
err := dit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
}
|
|
|
|
func TestCreateIndexTask_PreExecute(t *testing.T) {
|
|
collectionName := "collection1"
|
|
collectionID := UniqueID(1)
|
|
fieldName := newTestSchema().Fields[0].Name
|
|
|
|
Params.Init()
|
|
ic := newMockIndexCoord()
|
|
ctx := context.Background()
|
|
|
|
mockCache := newMockCache()
|
|
mockCache.setGetIDFunc(func(ctx context.Context, collectionName string) (typeutil.UniqueID, error) {
|
|
return collectionID, nil
|
|
})
|
|
mockCache.setGetSchemaFunc(func(ctx context.Context, collectionName string) (*schemapb.CollectionSchema, error) {
|
|
return newTestSchema(), nil
|
|
})
|
|
globalMetaCache = mockCache
|
|
|
|
cit := createIndexTask{
|
|
ctx: ctx,
|
|
req: &milvuspb.CreateIndexRequest{
|
|
Base: &commonpb.MsgBase{
|
|
MsgType: commonpb.MsgType_CreateIndex,
|
|
},
|
|
CollectionName: collectionName,
|
|
FieldName: fieldName,
|
|
},
|
|
indexCoord: ic,
|
|
queryCoord: nil,
|
|
result: nil,
|
|
collectionID: collectionID,
|
|
}
|
|
|
|
t.Run("normal", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
CollectionIDs: []int64{},
|
|
}, nil
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
cit.queryCoord = qc
|
|
|
|
err := cit.PreExecute(ctx)
|
|
assert.NoError(t, err)
|
|
})
|
|
|
|
t.Run("coll has been loaded", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
CollectionIDs: []int64{collectionID},
|
|
}, nil
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
cit.queryCoord = qc
|
|
err := cit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("check load error", func(t *testing.T) {
|
|
showCollectionMock := func(ctx context.Context, request *querypb.ShowCollectionsRequest) (*querypb.ShowCollectionsResponse, error) {
|
|
return &querypb.ShowCollectionsResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_UnexpectedError,
|
|
Reason: "fail reason",
|
|
},
|
|
CollectionIDs: nil,
|
|
}, errors.New("error")
|
|
}
|
|
qc := NewQueryCoordMock(withValidShardLeaders(), SetQueryCoordShowCollectionsFunc(showCollectionMock))
|
|
qc.updateState(commonpb.StateCode_Healthy)
|
|
cit.queryCoord = qc
|
|
err := cit.PreExecute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
}
|
|
|
|
func TestDropIndexTask_Execute(t *testing.T) {
|
|
collectionName := "collection1"
|
|
collectionID := UniqueID(1)
|
|
fieldID := UniqueID(2)
|
|
indexID := UniqueID(3)
|
|
fieldName := "field1"
|
|
indexName := "_default_idx_101"
|
|
|
|
ic := newMockIndexCoord()
|
|
ctx := context.Background()
|
|
|
|
mockCache := newMockCache()
|
|
mockCache.setGetIDFunc(func(ctx context.Context, collectionName string) (typeutil.UniqueID, error) {
|
|
return collectionID, nil
|
|
})
|
|
globalMetaCache = mockCache
|
|
|
|
ic.DropIndexFunc = func(ctx context.Context, request *indexpb.DropIndexRequest) (*commonpb.Status, error) {
|
|
return &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
}, nil
|
|
}
|
|
dit := dropIndexTask{
|
|
ctx: ctx,
|
|
DropIndexRequest: &milvuspb.DropIndexRequest{
|
|
CollectionName: collectionName,
|
|
FieldName: fieldName,
|
|
IndexName: "",
|
|
},
|
|
indexCoord: ic,
|
|
queryCoord: nil,
|
|
result: nil,
|
|
collectionID: collectionID,
|
|
}
|
|
|
|
t.Run("one index", func(t *testing.T) {
|
|
ic.DescribeIndexFunc = func(ctx context.Context, request *indexpb.DescribeIndexRequest) (*indexpb.DescribeIndexResponse, error) {
|
|
return &indexpb.DescribeIndexResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
IndexInfos: []*indexpb.IndexInfo{
|
|
{
|
|
CollectionID: collectionID,
|
|
FieldID: fieldID,
|
|
IndexName: indexName,
|
|
IndexID: indexID,
|
|
TypeParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "dim",
|
|
Value: "128",
|
|
},
|
|
},
|
|
IndexParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "metrics_type",
|
|
Value: "L2",
|
|
},
|
|
},
|
|
IndexedRows: 10000,
|
|
TotalRows: 10000,
|
|
State: commonpb.IndexState_Finished,
|
|
IndexStateFailReason: "",
|
|
IsAutoIndex: false,
|
|
UserIndexParams: nil,
|
|
},
|
|
},
|
|
}, nil
|
|
}
|
|
err := dit.Execute(ctx)
|
|
assert.NoError(t, err)
|
|
})
|
|
|
|
t.Run("two indexes", func(t *testing.T) {
|
|
ic.DescribeIndexFunc = func(ctx context.Context, request *indexpb.DescribeIndexRequest) (*indexpb.DescribeIndexResponse, error) {
|
|
return &indexpb.DescribeIndexResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_Success,
|
|
},
|
|
IndexInfos: []*indexpb.IndexInfo{
|
|
{
|
|
CollectionID: collectionID,
|
|
FieldID: fieldID,
|
|
IndexName: indexName,
|
|
IndexID: indexID,
|
|
TypeParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "dim",
|
|
Value: "128",
|
|
},
|
|
},
|
|
IndexParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "metrics_type",
|
|
Value: "L2",
|
|
},
|
|
{
|
|
Key: "index_type",
|
|
Value: "IVF_FLAT",
|
|
},
|
|
},
|
|
IndexedRows: 10000,
|
|
TotalRows: 10000,
|
|
State: commonpb.IndexState_Finished,
|
|
IndexStateFailReason: "",
|
|
IsAutoIndex: false,
|
|
UserIndexParams: nil,
|
|
},
|
|
{
|
|
CollectionID: collectionID,
|
|
FieldID: fieldID + 1,
|
|
IndexName: "_default_idx_102",
|
|
IndexID: indexID + 1,
|
|
TypeParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "dim",
|
|
Value: "128",
|
|
},
|
|
},
|
|
IndexParams: []*commonpb.KeyValuePair{
|
|
{
|
|
Key: "metrics_type",
|
|
Value: "L2",
|
|
},
|
|
{
|
|
Key: "index_type",
|
|
Value: "FLAT",
|
|
},
|
|
},
|
|
IndexedRows: 10000,
|
|
TotalRows: 10000,
|
|
State: commonpb.IndexState_Finished,
|
|
IndexStateFailReason: "",
|
|
IsAutoIndex: false,
|
|
UserIndexParams: nil,
|
|
},
|
|
},
|
|
}, nil
|
|
}
|
|
dit.indexCoord = ic
|
|
dit.IndexName = ""
|
|
|
|
err := dit.Execute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("describe index error", func(t *testing.T) {
|
|
ic.DescribeIndexFunc = func(ctx context.Context, request *indexpb.DescribeIndexRequest) (*indexpb.DescribeIndexResponse, error) {
|
|
return nil, errors.New("error")
|
|
}
|
|
dit.indexCoord = ic
|
|
dit.IndexName = ""
|
|
|
|
err := dit.Execute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("describe index fail", func(t *testing.T) {
|
|
ic.DescribeIndexFunc = func(ctx context.Context, request *indexpb.DescribeIndexRequest) (*indexpb.DescribeIndexResponse, error) {
|
|
return &indexpb.DescribeIndexResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_UnexpectedError,
|
|
Reason: "fail reason",
|
|
},
|
|
IndexInfos: nil,
|
|
}, nil
|
|
}
|
|
dit.indexCoord = ic
|
|
dit.IndexName = ""
|
|
|
|
err := dit.Execute(ctx)
|
|
assert.Error(t, err)
|
|
})
|
|
|
|
t.Run("index not exist", func(t *testing.T) {
|
|
ic.DescribeIndexFunc = func(ctx context.Context, request *indexpb.DescribeIndexRequest) (*indexpb.DescribeIndexResponse, error) {
|
|
return &indexpb.DescribeIndexResponse{
|
|
Status: &commonpb.Status{
|
|
ErrorCode: commonpb.ErrorCode_IndexNotExist,
|
|
Reason: "index not exist",
|
|
},
|
|
IndexInfos: nil,
|
|
}, nil
|
|
}
|
|
dit.indexCoord = ic
|
|
dit.IndexName = ""
|
|
|
|
err := dit.Execute(ctx)
|
|
assert.NoError(t, err)
|
|
})
|
|
}
|