search

milvus-io / milvus-sdk-cpp

C++ SDK for Milvus 2.0
Apache License 2.0
38 stars 21 forks source link

Refine types header. #171

Closed matrixji closed 2 years ago

matrixji commented 2 years ago

fixes #168

Signed-off-by: Ji Bin matrixji@live.com

sre-ci-robot commented 2 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matrixji

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: - ~~[OWNERS](https://github.com/milvus-io/milvus-sdk-cpp/blob/master/OWNERS)~~ [matrixji] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
matrixji commented 2 years ago

/cc @yhmo /cc @ArkaprabhaChakraborty

codecov[bot] commented 2 years ago

Codecov Report

Merging #171 (4490b5d) into master (504b025) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #171      +/-   ##
==========================================
+ Coverage   99.26%   99.30%   +0.04%     
==========================================
  Files          44       47       +3     
  Lines        2298     2315      +17     
==========================================
+ Hits         2281     2299      +18     
+ Misses         17       16       -1
Impacted Files Coverage Δ
src/impl/types/SearchArguments.cpp 100.00% <ø> (ø)
src/include/milvus/types/HybirdTimestamp.h 100.00% <ø> (ø)
src/include/milvus/types/ProgressMonitor.h 100.00% <ø> (ø)
src/include/milvus/types/QueryResults.h 100.00% <ø> (ø)
src/include/milvus/types/SearchResults.h 100.00% <ø> (ø)
src/impl/types/HybirdTimestamp.cpp 100.00% <100.00%> (ø)
src/impl/types/IDArray.cpp 100.00% <100.00%> (ø)
src/impl/types/IndexState.cpp 100.00% <100.00%> (ø)
src/impl/types/PartitionInfo.cpp 100.00% <100.00%> (ø)
src/impl/types/PartitionStat.cpp 100.00% <100.00%> (ø)
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 504b025...4490b5d. Read the comment docs.

yhmo commented 2 years ago

/lgtm