Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: Wrong data type when import array by bulkwriter #2034

Open
1 task done
bigsheeper opened this issue Apr 11, 2024 · 1 comment
Open
1 task done

[Bug]: Wrong data type when import array by bulkwriter #2034

bigsheeper opened this issue Apr 11, 2024 · 1 comment
Assignees
@bigsheeper
Labels
kind/bug Something isn't working

Comments

@bigsheeper
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

The float32 array intended for import via the bulk writer was imported as double instead.
For the float32 type, we should specify "datatype=float32" just like float vectors; otherwise, it will default to double.

Expected Behavior

No response

Steps/Code To Reproduce behavior

No response

Environment details

- Hardware/Softward conditions (OS, CPU, GPU, Memory):
- Method of installation (Docker, or from source):
- Milvus version (v0.3.1, or v0.4.0):
- Milvus configuration (Settings you made in `server_config.yaml`):

Anything else?

No response
@bigsheeper bigsheeper added the kind/bug Something isn't working label Apr 11, 2024
@bigsheeper
Copy link
Contributor Author

/assign

This was referenced Apr 11, 2024
Merged
Merged
sre-ci-robot pushed a commit that referenced this issue Apr 12, 2024
@bigsheeper
fix: Fix import array via bulkwriter (#2035) (#2036)
64acef6 
When import via bulkwriter, for array type, we should specify datatype;
otherwise, it will default to double(for float32 array) and int64(for
int8, int16, int32 array).

issue: #2034,
milvus-io/milvus#31834

pr: #2035

Signed-off-by: bigsheeper <[email protected]>
sre-ci-robot pushed a commit that referenced this issue Apr 12, 2024
@bigsheeper
fix: Fix import array via bulkwriter (#2035)
989d15b 
When import via bulkwriter, for array type, we should specify datatype;
otherwise, it will default to double(for float32 array) and int64(for
int8, int16, int32 array).

issue: #2034,
milvus-io/milvus#31834

Signed-off-by: bigsheeper <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bigsheeper bigsheeper

Labels
kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bigsheeper