Skip to content

Comments

refactor: rm boost#3397

Merged
dl239 merged 6 commits into4paradigm:mainfrom
emo-coder:refactor/rm_boost
Jul 31, 2023
Merged

refactor: rm boost#3397
dl239 merged 6 commits into4paradigm:mainfrom
emo-coder:refactor/rm_boost

Conversation

@emo-coder
Copy link
Contributor

  • 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)?

@github-actions github-actions bot added execute-engine hybridse sql engine storage-engine openmldb storage engine. nameserver & tablet labels Jul 25, 2023
@emo-coder emo-coder force-pushed the refactor/rm_boost branch from 8a50cf1 to 5de59a7 Compare July 26, 2023 03:30
@github-actions github-actions bot removed the execute-engine hybridse sql engine label Jul 26, 2023
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01% 🎉

Comparison is base (c4a8371) 75.70% compared to head (d3f9de9) 75.71%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3397      +/-   ##
============================================
+ Coverage     75.70%   75.71%   +0.01%     
  Complexity      414      414              
============================================
  Files           700      700              
  Lines        126668   126668              
  Branches       1206     1206              
============================================
+ Hits          95888    95906      +18     
+ Misses        30539    30521      -18     
  Partials        241      241              
Files Changed Coverage Δ
src/sdk/file_option_parser.h 90.35% <100.00%> (ø)
src/sdk/sql_cluster_router.cc 61.07% <100.00%> (+0.07%) ⬆️

... and 7 files with indirect coverage changes

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

@dl239 dl239 marked this pull request as ready for review July 26, 2023 07:52
@dl239 dl239 enabled auto-merge (squash) July 31, 2023 07:55
@dl239 dl239 merged commit 4286d79 into 4paradigm:main Jul 31, 2023
dl239 pushed a commit that referenced this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

storage-engine openmldb storage engine. nameserver & tablet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants