Skip to content

Comments

fix: add require check message#3638

Merged
vagetablechicken merged 1 commit into4paradigm:mainfrom
vagetablechicken:sfix
Dec 8, 2023
Merged

fix: add require check message#3638
vagetablechicken merged 1 commit into4paradigm:mainfrom
vagetablechicken:sfix

Conversation

@vagetablechicken
Copy link
Collaborator

No description provided.

@github-actions github-actions bot added the batch-engine openmldb batch(offline) engine label Nov 29, 2023
@codecov
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (5e225da) 40.16% compared to head (d9486dd) 74.96%.
Report is 2 commits behind head on main.

Files Patch % Lines
..._4paradigm/openmldb/batch/utils/HybridseUtil.scala 50.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3638       +/-   ##
=============================================
+ Coverage     40.16%   74.96%   +34.80%     
  Complexity      640      640               
=============================================
  Files           191      724      +533     
  Lines         11233   130383   +119150     
  Branches       1325     1332        +7     
=============================================
+ Hits           4512    97747    +93235     
- Misses         6428    32343    +25915     
  Partials        293      293               

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vagetablechicken vagetablechicken merged commit c081f28 into 4paradigm:main Dec 8, 2023
@vagetablechicken vagetablechicken deleted the sfix branch December 8, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

batch-engine openmldb batch(offline) engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants