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.

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

fix can't insert string == schema max len #1304

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

izapolsk
Copy link
Contributor

a quick fix for bug #1143

@XuanYang-cn
Copy link
Contributor

@izapolsk Please sign off this PR following the DCO robots' instruction https://github.com/milvus-io/pymilvus/pull/1304/checks?check_run_id=11322034734

@izapolsk izapolsk force-pushed the fix/varchar-max-len branch 2 times, most recently from d0a7355 to cf23dbc Compare February 15, 2023 06:58
@izapolsk
Copy link
Contributor Author

@XuanYang-cn, done, thank you

…with len same as varchar max len.

Signed-off-by: Ievgen Zapolskyi <[email protected]>
@izapolsk izapolsk force-pushed the fix/varchar-max-len branch from cf23dbc to 6fbccb7 Compare February 15, 2023 07:01
@mergify mergify bot added dco-passed and removed needs-dco labels Feb 15, 2023
@XuanYang-cn
Copy link
Contributor

@izapolsk Thanks for this contribution
/lgtm
/approve

@sre-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: izapolsk, XuanYang-cn

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

The pull request process is described here

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

@sre-ci-robot sre-ci-robot merged commit 061c20e into milvus-io:master Feb 15, 2023
@izapolsk izapolsk deleted the fix/varchar-max-len branch February 15, 2023 08:53
XuanYang-cn pushed a commit to XuanYang-cn/pymilvus that referenced this pull request Mar 1, 2023
- fix describe/list rg err msg (milvus-io#1291)
- fix load collection (milvus-io#1293)
- Change default connect timeout to 10s (milvus-io#1294)
- add param check (milvus-io#1297)
- Support rename collection API (milvus-io#1299)
- expose rg info in replica (milvus-io#1305)
- fix check string max len (milvus-io#1304)
- Support upsert (milvus-io#1303)
- expose rg info (milvus-io#1307)
- Add ignore_growing option for query and search (milvus-io#1286)
- refine rg api (milvus-io#1310)
- Fix some error (milvus-io#1312)
- Add GPU index types (milvus-io#1314)

Signed-off-by: yangxuan <[email protected]>

Co-authored-by: Wei Liu <[email protected]>
Co-authored-by: jaime <[email protected]>
Co-authored-by: Ievgen Zapolskyi <[email protected]>
Co-authored-by: lixinguo <[email protected]>
Co-authored-by: aoiasd <[email protected]>
Co-authored-by: Yudong Cai <[email protected]>
sre-ci-robot pushed a commit that referenced this pull request Mar 1, 2023
- fix describe/list rg err msg (#1291)
- fix load collection (#1293)
- Change default connect timeout to 10s (#1294)
- add param check (#1297)
- Support rename collection API (#1299)
- expose rg info in replica (#1305)
- fix check string max len (#1304)
- Support upsert (#1303)
- expose rg info (#1307)
- Add ignore_growing option for query and search (#1286)
- refine rg api (#1310)
- Fix some error (#1312)
- Add GPU index types (#1314)

Co-authored-by: wei liu <[email protected]>
Co-authored-by: jaime <[email protected]>
Co-authored-by: Ievgen Zapolskyi <[email protected]>
Co-authored-by: lixinguo <[email protected]>
Co-authored-by: aoiasd <[email protected]>
Co-authored-by: Yudong Cai <[email protected]>
XuanYang-cn pushed a commit to XuanYang-cn/pymilvus that referenced this pull request Mar 7, 2023
…with len same as varchar max len. (milvus-io#1304)

Signed-off-by: Ievgen Zapolskyi <[email protected]>
XuanYang-cn pushed a commit to XuanYang-cn/pymilvus that referenced this pull request Mar 7, 2023
Otherwise it's impossible to insert string with len same as varchar max len. (milvus-io#1304)

Co-authored-by: Ievgen Zapolskyi <[email protected]>
Signed-off-by: yangxuan <[email protected]>
@milvus-io milvus-io locked as resolved and limited conversation to collaborators Mar 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants