Files
milvus/tests/integration/import/commit_timestamp_test.go
XuanYang-cn f36cf77d79 test: reduce integration import and shared wait overhead (#53525)
## 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]>
2026-09-18 17:07:33 +08:00

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))
}