milvus icon indicating copy to clipboard operation
milvus copied to clipboard

fix: [restful v2] bug list

Open PowderLi opened this issue 1 year ago • 4 comments

bug list

  1. element data type is needed while create a collection with an array field #30638
  2. spelling mistake about metricsType #30643
  3. new collection enable dynamic field / auto id as user defined #30665
  4. convert rowCount from string to int #30661
  5. try it's best to create a new collection #30652
  6. int64 percious #20415
  7. insert into collection which has multi vector fields #30674
  8. cannot rename a collection to other database #30700
  9. update the request body of "indexes/create" #30769
  10. got [] while list indexes of a collection which has no index #30722
  11. restful need encode password before call CreateCredential/UpdateCredential #30730
  12. some parameters missed the required label #30737 13.define the field to be or not to be a partition key while create a collection #30797

enhance: support opentelemetry enhance: support dataType: Float16Vector & BFloat16Vector #22837 enhance: describe collection will show the field is partition key or not #30789

PowderLi avatar Feb 21 '24 12:02 PowderLi

@PowderLi ut workflow job failed, comment rerun ut can trigger the job again.

mergify[bot] avatar Feb 21 '24 13:02 mergify[bot]

Codecov Report

Attention: Patch coverage is 83.03571% with 38 lines in your changes are missing coverage. Please review.

Project coverage is 80.47%. Comparing base (0661d33) to head (53758b2). Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #30740      +/-   ##
==========================================
- Coverage   80.72%   80.47%   -0.26%     
==========================================
  Files         966      966              
  Lines      137289   137412     +123     
==========================================
- Hits       110832   110578     -254     
- Misses      22730    23100     +370     
- Partials     3727     3734       +7     
Files Coverage Δ
...nternal/distributed/proxy/httpserver/handler_v1.go 90.90% <100.00%> (+0.01%) :arrow_up:
...nternal/distributed/proxy/httpserver/request_v2.go 100.00% <100.00%> (ø)
...nternal/distributed/proxy/httpserver/handler_v2.go 89.66% <96.29%> (+0.36%) :arrow_up:
internal/distributed/proxy/httpserver/utils.go 84.04% <74.07%> (-0.54%) :arrow_down:

... and 32 files with indirect coverage changes

codecov[bot] avatar Feb 21 '24 15:02 codecov[bot]

@PowderLi E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

mergify[bot] avatar Feb 24 '24 07:02 mergify[bot]

@PowderLi E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

mergify[bot] avatar Feb 27 '24 03:02 mergify[bot]

/run-cpu-e2e

PowderLi avatar Feb 27 '24 05:02 PowderLi

/approve /lgtm

czs007 avatar Feb 27 '24 08:02 czs007

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: czs007, PowderLi

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 avatar Feb 27 '24 08:02 sre-ci-robot