mirror of
https://github.com/milvus-io/milvus.git
synced 2026-10-12 02:19:10 +00:00
## What changed Integration tests spend substantial time waiting for scheduler ticks and repeating collection/import/index/load setup. This PR reduces that overhead while preserving the existing data volumes, supported format combinations, and completion/failure assertions. - Shorten import scheduling and shared readiness polling intervals, and reduce collection teardown overhead. - Group vector imports by vector type: 23 setup workflows become 6, retaining all 23 vector/format combinations and 2,300 rows, with per-format query and search checks. - Combine the four dynamic-field formats into one collection/import job, retaining 400 rows and per-format validation. - Fix binlog-import error classification: invalid backup path counts remain non-retryable `ErrImportFailed` input errors; other listing failures remain service-unavailable errors with their causes preserved. - Replace QueryNode fixed sleeps with readiness checks, and make BloomMatch recall comparisons use identical query vectors. Scheduler changes are test-only. The binlog error-classification fix changes production behavior and has regression coverage. ## Performance and validation The measured head `a250d9c63429` completed the full IT run successfully. Compare the original baseline [PR #53168 / IT #14635](https://jenkins-milvus-ci.milvus.io/job/MILVUS-CI-V2-PR-PIPELINES/job/milvus-ut-integration-pipeline/14635/) with [this PR / IT #14690](https://jenkins-milvus-ci.milvus.io/job/MILVUS-CI-V2-PR-PIPELINES/job/milvus-ut-integration-pipeline/14690/): **Plain IT runtime (Integration Tests stage): 87m21s → 62m16s, saving 25m04s (28.7%).** Package logs confirm the same **45 packages** completed: 44 passed and one had no test files. The final run recorded **427 passing test events, seven skips, and zero failures**. Existing focused checks also passed for invalid binlog inputs, storage-error handling, dynamic-field formats, and the complete vector format matrix. These are measured historical CI runs; machine and run-to-run variation still applies. Displayed times are rounded independently, with reductions calculated from the original seconds. --------- Signed-off-by: yangxuan <[email protected]>
155 lines
5.2 KiB
Go
155 lines
5.2 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 importv2
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/suite"
|
|
"google.golang.org/protobuf/proto"
|
|
|
|
"github.com/milvus-io/milvus-proto/go-api/v3/commonpb"
|
|
"github.com/milvus-io/milvus-proto/go-api/v3/milvuspb"
|
|
"github.com/milvus-io/milvus/pkg/v3/common"
|
|
"github.com/milvus-io/milvus/pkg/v3/proto/internalpb"
|
|
"github.com/milvus-io/milvus/pkg/v3/util/funcutil"
|
|
"github.com/milvus-io/milvus/pkg/v3/util/merr"
|
|
"github.com/milvus-io/milvus/tests/integration"
|
|
)
|
|
|
|
type CommitTimestampSuite struct {
|
|
importSuite
|
|
}
|
|
|
|
// TestImport_CommitTimestampSetAfterCompletion verifies that all segments produced
|
|
// by a bulk-insert import have CommitTimestamp > 0 once the job completes.
|
|
func (s *CommitTimestampSuite) TestImport_CommitTimestampSetAfterCompletion() {
|
|
const rowCount = 100
|
|
|
|
c := s.Cluster
|
|
ctx, cancel := context.WithTimeout(c.GetContext(), 240*time.Second)
|
|
defer cancel()
|
|
|
|
collectionName := "TestCommitTS_" + funcutil.RandomString(8)
|
|
|
|
schema := integration.ConstructSchema(collectionName, dim, false)
|
|
marshaledSchema, err := proto.Marshal(schema)
|
|
s.NoError(err)
|
|
|
|
createCollectionStatus, err := c.MilvusClient.CreateCollection(ctx, &milvuspb.CreateCollectionRequest{
|
|
CollectionName: collectionName,
|
|
Schema: marshaledSchema,
|
|
ShardsNum: common.DefaultShardsNum,
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(commonpb.ErrorCode_Success, createCollectionStatus.GetErrorCode())
|
|
|
|
filePath, err := GenerateParquetFile(s.Cluster, schema, rowCount)
|
|
s.NoError(err)
|
|
|
|
importResp, err := c.ProxyClient.ImportV2(ctx, &internalpb.ImportRequest{
|
|
CollectionName: collectionName,
|
|
Files: []*internalpb.ImportFile{
|
|
{Paths: []string{filePath}},
|
|
},
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(int32(0), importResp.GetStatus().GetCode())
|
|
|
|
err = WaitForImportDone(ctx, c, importResp.GetJobID())
|
|
s.NoError(err)
|
|
|
|
AssertImportSegmentsHaveCommitTimestamp(s.T(), c, collectionName)
|
|
}
|
|
|
|
// TestImport_DataQueryableAfterCommit verifies that a Strong-consistency query
|
|
// returns all imported rows once the import job has completed.
|
|
func (s *CommitTimestampSuite) TestImport_DataQueryableAfterCommit() {
|
|
const rowCount = 100
|
|
|
|
c := s.Cluster
|
|
ctx, cancel := context.WithTimeout(c.GetContext(), 300*time.Second)
|
|
defer cancel()
|
|
|
|
collectionName := "TestCommitTSQuery_" + funcutil.RandomString(8)
|
|
|
|
schema := integration.ConstructSchema(collectionName, dim, false)
|
|
marshaledSchema, err := proto.Marshal(schema)
|
|
s.NoError(err)
|
|
|
|
createCollectionStatus, err := c.MilvusClient.CreateCollection(ctx, &milvuspb.CreateCollectionRequest{
|
|
CollectionName: collectionName,
|
|
Schema: marshaledSchema,
|
|
ShardsNum: common.DefaultShardsNum,
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(commonpb.ErrorCode_Success, createCollectionStatus.GetErrorCode())
|
|
|
|
filePath, err := GenerateParquetFile(s.Cluster, schema, rowCount)
|
|
s.NoError(err)
|
|
|
|
importResp, err := c.ProxyClient.ImportV2(ctx, &internalpb.ImportRequest{
|
|
CollectionName: collectionName,
|
|
Files: []*internalpb.ImportFile{
|
|
{Paths: []string{filePath}},
|
|
},
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(int32(0), importResp.GetStatus().GetCode())
|
|
|
|
err = WaitForImportDone(ctx, c, importResp.GetJobID())
|
|
s.NoError(err)
|
|
|
|
// create index on the vector field
|
|
createIndexStatus, err := c.MilvusClient.CreateIndex(ctx, &milvuspb.CreateIndexRequest{
|
|
CollectionName: collectionName,
|
|
FieldName: integration.FloatVecField,
|
|
IndexName: "_default",
|
|
ExtraParams: integration.ConstructIndexParam(dim, "HNSW", "L2"),
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(commonpb.ErrorCode_Success, createIndexStatus.GetErrorCode())
|
|
|
|
s.WaitForIndexBuilt(ctx, collectionName, integration.FloatVecField)
|
|
|
|
// load collection
|
|
loadStatus, err := c.MilvusClient.LoadCollection(ctx, &milvuspb.LoadCollectionRequest{
|
|
CollectionName: collectionName,
|
|
})
|
|
s.NoError(err)
|
|
s.Equal(commonpb.ErrorCode_Success, loadStatus.GetErrorCode())
|
|
s.WaitForLoad(ctx, collectionName)
|
|
|
|
// Strong-consistency count(*) query — must see all imported rows
|
|
queryResult, err := c.MilvusClient.Query(ctx, &milvuspb.QueryRequest{
|
|
CollectionName: collectionName,
|
|
Expr: integration.Int64Field + " >= 0",
|
|
OutputFields: []string{"count(*)"},
|
|
ConsistencyLevel: commonpb.ConsistencyLevel_Strong,
|
|
})
|
|
s.NoError(merr.CheckRPCCall(queryResult, err))
|
|
count := int(queryResult.GetFieldsData()[0].GetScalars().GetLongData().GetData()[0])
|
|
s.Equal(rowCount, count,
|
|
"Strong-consistency query should return all %d imported rows, got %d", rowCount, count)
|
|
}
|
|
|
|
func TestCommitTimestampSuite(t *testing.T) {
|
|
suite.Run(t, new(CommitTimestampSuite))
|
|
}
|