Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: upgrade thirdparty to 0.6.0 #3557

Merged
merged 4 commits into from
Oct 27, 2023

Conversation

aceforeverd
Copy link
Collaborator

@aceforeverd aceforeverd commented Oct 17, 2023

  1. upgrade thirdparty, which upgrade brpc to 1.6.0
  2. calls some teardown function before nameserver destructor, it mitigate cases like pure virtual method called #3416

it upgrade brpc to 1.6.0
@github-actions github-actions bot added build openmldb compiling and installing docker openmldb compile image or demo image labels Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (88e9e79) 83.35% compared to head (f4b478e) 75.30%.
Report is 8 commits behind head on main.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3557       +/-   ##
=============================================
- Coverage     83.35%   75.30%    -8.05%     
- Complexity        0      587      +587     
=============================================
  Files            17      718      +701     
  Lines           751   129787   +129036     
  Branches          0     1280     +1280     
=============================================
+ Hits            626    97737    +97111     
- Misses          109    31764    +31655     
- Partials         16      286      +270     
Files Coverage Δ
src/nameserver/name_server_impl.h 100.00% <100.00%> (ø)
src/sdk/mini_cluster.h 79.76% <100.00%> (ø)

... and 699 files with indirect coverage changes

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

@aceforeverd aceforeverd force-pushed the build-thirdparty-0.6.0 branch from e6193ea to bfa81f2 Compare October 18, 2023 02:17
@github-actions github-actions bot added the storage-engine openmldb storage engine. nameserver & tablet label Oct 23, 2023
@aceforeverd aceforeverd force-pushed the build-thirdparty-0.6.0 branch from 5d32ca7 to 776af30 Compare October 23, 2023 05:59
@aceforeverd aceforeverd requested review from dl239, vagetablechicken, tobegit3hub and lqy222 and removed request for dl239 October 23, 2023 05:59
Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aceforeverd aceforeverd force-pushed the build-thirdparty-0.6.0 branch from 776af30 to f4b478e Compare October 26, 2023 09:25
@aceforeverd aceforeverd requested a review from dl239 October 26, 2023 11:33
@aceforeverd aceforeverd merged commit 5c6b40c into 4paradigm:main Oct 27, 2023
@aceforeverd aceforeverd deleted the build-thirdparty-0.6.0 branch October 27, 2023 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build openmldb compiling and installing docker openmldb compile image or demo image storage-engine openmldb storage engine. nameserver & tablet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants