Skip to content

BulkWriter support binary/float16/int8 vectors in Struct field#1932

Merged
sre-ci-robot merged 1 commit into
milvus-io:masterfrom
yhmo:ma
Jun 26, 2026
Merged

BulkWriter support binary/float16/int8 vectors in Struct field#1932
sre-ci-robot merged 1 commit into
milvus-io:masterfrom
yhmo:ma

Conversation

@yhmo

@yhmo yhmo commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@sre-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yhmo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mergify

mergify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@mergify mergify Bot added the dco-passed label Jun 26, 2026
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 53.33333% with 21 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (master@46b3538). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...io/milvus/bulkwriter/common/utils/WriterUtils.java 66.66% 6 Missing and 3 partials ⚠️
.../bulkwriter/storage/client/MinioStorageClient.java 0.00% 9 Missing ⚠️
...in/java/io/milvus/bulkwriter/RemoteBulkWriter.java 0.00% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1932   +/-   ##
=========================================
  Coverage          ?   62.85%           
  Complexity        ?     3921           
=========================================
  Files             ?      457           
  Lines             ?    25362           
  Branches          ?     2324           
=========================================
  Hits              ?    15941           
  Misses            ?     8115           
  Partials          ?     1306           
Files with missing lines Coverage Δ
...o/milvus/bulkwriter/common/utils/ParquetUtils.java 83.17% <100.00%> (ø)
...ava/io/milvus/bulkwriter/writer/CSVFileWriter.java 89.36% <100.00%> (ø)
...va/io/milvus/bulkwriter/writer/JSONFileWriter.java 87.09% <100.00%> (ø)
...in/java/io/milvus/bulkwriter/RemoteBulkWriter.java 0.00% <0.00%> (ø)
...io/milvus/bulkwriter/common/utils/WriterUtils.java 66.66% <66.66%> (ø)
.../bulkwriter/storage/client/MinioStorageClient.java 16.41% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread sdk-bulkwriter/src/main/java/io/milvus/bulkwriter/writer/CSVFileWriter.java Outdated
Comment thread sdk-bulkwriter/src/main/java/io/milvus/bulkwriter/writer/JSONFileWriter.java Outdated
Comment thread sdk-bulkwriter/src/main/java/io/milvus/bulkwriter/writer/CSVFileWriter.java Outdated
Signed-off-by: yhmo <yihua.mo@zilliz.com>
@sre-ci-robot sre-ci-robot merged commit 831f1b1 into milvus-io:master Jun 26, 2026
7 of 8 checks passed
@yhmo yhmo deleted the ma branch June 29, 2026 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants