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

fix: join brpc server after stop it #2173

Conversation

mangoGoForward
Copy link
Contributor

Signed-off-by: mango [email protected]

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

  • What is the current behavior? (You can also link to an open issue here)
    join brpc server after stop it
    fix bug: we should join after stop brpc server #2157

  • What is the new behavior (if this is a feature change)?
    None

@github-actions github-actions bot added the storage-engine openmldb storage engine. nameserver & tablet label Jul 14, 2022
@codecov
Copy link

codecov bot commented Jul 14, 2022

Codecov Report

Merging #2173 (a43226b) into main (467e231) will increase coverage by 38.66%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##               main    #2173       +/-   ##
=============================================
+ Coverage     37.09%   75.75%   +38.66%     
  Complexity      353      353               
=============================================
  Files           143      619      +476     
  Lines          8557   117271   +108714     
  Branches       1039     1039               
=============================================
+ Hits           3174    88837    +85663     
- Misses         5172    28223    +23051     
  Partials        211      211               
Impacted Files Coverage Δ
src/sdk/mini_cluster.h 79.26% <100.00%> (ø)
hybridse/include/base/raw_buffer.h 12.50% <0.00%> (ø)
hybridse/include/base/sig_trace.h 0.00% <0.00%> (ø)
hybridse/src/planv2/planner_v2.h 100.00% <0.00%> (ø)
hybridse/src/vm/mem_catalog.cc 73.33% <0.00%> (ø)
hybridse/src/passes/lambdafy_projects_test.cc 100.00% <0.00%> (ø)
hybridse/include/base/fe_object.h 100.00% <0.00%> (ø)
...ics/query_response_time/query_response_time_util.h 100.00% <0.00%> (ø)
hybridse/include/sdk/codec_sdk.h 96.87% <0.00%> (ø)
hybridse/src/case/case_data_mock_test.cc 100.00% <0.00%> (ø)
... and 467 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 467e231...a43226b. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 14, 2022

SDK Test Report

  77 files    77 suites   6m 37s ⏱️
165 tests 163 ✔️ 2 💤 0
206 runs  204 ✔️ 2 💤 0

Results for commit a43226b.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 14, 2022

Linux Test Report

       55 files       200 suites   53m 8s ⏱️
  9 229 tests   9 225 ✔️ 4 💤 0
13 580 runs  13 576 ✔️ 4 💤 0

Results for commit a43226b.

♻️ This comment has been updated with latest results.

@dl239 dl239 requested a review from vagetablechicken July 15, 2022 01:06
@dl239 dl239 enabled auto-merge (squash) July 15, 2022 01:06
auto-merge was automatically disabled July 15, 2022 06:21

Head branch was pushed to by a user without write access

@mangoGoForward mangoGoForward force-pushed the fix/join-brpc-server-after-stop branch from 152e10d to 46a041b Compare July 15, 2022 06:21
@mangoGoForward mangoGoForward force-pushed the fix/join-brpc-server-after-stop branch from 46a041b to a43226b Compare July 15, 2022 07:53
@vagetablechicken vagetablechicken merged commit 819fb69 into 4paradigm:main Jul 15, 2022
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.

bug: we should join after stop brpc server
3 participants