OpenMLDB icon indicating copy to clipboard operation
OpenMLDB copied to clipboard

feat(go-sdk): init go sdk

Open qsliu2017 opened this issue 1 year ago • 8 comments

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

  • What is the current behavior? (You can also link to an open issue here)

  • What is the new behavior (if this is a feature change)? add a no-op go sql driver

qsliu2017 avatar Jul 24 '22 13:07 qsliu2017

Codecov Report

Base: 35.97% // Head: 75.82% // Increases project coverage by +39.85% :tada:

Coverage data is based on head (7f0c2af) compared to base (242c512). Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #2232       +/-   ##
=============================================
+ Coverage     35.97%   75.82%   +39.85%     
- Complexity      371      377        +6     
=============================================
  Files           154      638      +484     
  Lines          9024   120492   +111468     
  Branches       1052     1053        +1     
=============================================
+ Hits           3246    91365    +88119     
- Misses         5561    28912    +23351     
+ Partials        217      215        -2     
Impacted Files Coverage Δ
src/apiserver/json_helper.cc 0.00% <0.00%> (ø)
src/apiserver/json_helper.h 0.00% <ø> (ø)
python/openmldb_sdk/tests/sdk_smoke_test.py 98.52% <0.00%> (-0.05%) :arrow_down:
python/openmldb_sdk/tests/dbapi_test.py 98.18% <0.00%> (ø)
python/openmldb_sdk/tests/sql_magic_test.py 78.72% <0.00%> (ø)
python/openmldb_sdk/tests/sqlalchemy_api_test.py 97.67% <0.00%> (ø)
...adigm/openmldb/taskmanager/zk/FailoverWatcher.java 0.00% <0.00%> (ø)
hybridse/src/vm/jit_runtime.h 100.00% <0.00%> (ø)
src/tablet/tablet_impl_multi_path_test.cc 100.00% <0.00%> (ø)
src/apiserver/interface_provider.cc 0.00% <0.00%> (ø)
... and 484 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jul 24 '22 13:07 codecov[bot]

SDK Test Report

  81 files    81 suites   7m 56s :stopwatch: 169 tests 167 :heavy_check_mark: 2 :zzz: 0 :x: 212 runs  210 :heavy_check_mark: 2 :zzz: 0 :x:

Results for commit 7f0c2af2.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Jul 24 '22 14:07 github-actions[bot]

Linux Test Report

561 tests   554 :heavy_check_mark:  26m 11s :stopwatch:   57 suites      7 :zzz:   51 files        0 :x:

Results for commit 7f0c2af2.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Jul 24 '22 14:07 github-actions[bot]

@qsliu2017 maybe you need some example code and test cases first ( readme and how to use it ) ? We can checking the interface first.

aceforeverd avatar Aug 26 '22 03:08 aceforeverd

you may consider data types in go. go has the ColumnType

aceforeverd avatar Sep 02 '22 08:09 aceforeverd

@qsliu2017 https://github.com/4paradigm/OpenMLDB/actions/runs/3102510202/jobs/5024874140#step:9:43 tests failed ?

aceforeverd avatar Sep 22 '22 07:09 aceforeverd

@qsliu2017 4paradigm/OpenMLDB/actions/runs/3102510202/jobs/5024874140#step:9:43 tests failed ?

addressed

qsliu2017 avatar Sep 22 '22 12:09 qsliu2017