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

Chore(tonic-build): set tonic-build.build_server(false), do not build Server code. #22

Merged
merged 3 commits into from
Jun 25, 2022

Conversation

CherishCai
Copy link
Contributor

@CherishCai CherishCai commented Jun 25, 2022

issue apache/skywalking#9271

the trace_segment_report_service_server::TraceSegmentReportServiceServer built by tonic-build is redundant now, and adding .build_server(false) can reduce the generation of this code.

@wu-sheng wu-sheng added the chore Housekeeping things like CI, license things label Jun 25, 2022
@wu-sheng wu-sheng added this to the 0.2.0 milestone Jun 25, 2022
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

Once e2e pass.

@wu-sheng
Copy link
Member

Your codes are not formated correctly, please fix it.

@codecov-commenter
Copy link

Codecov Report

Merging #22 (9b5fd67) into master (726e428) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   86.91%   86.91%           
=======================================
  Files           9        9           
  Lines         298      298           
=======================================
  Hits          259      259           
  Misses         39       39           

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 726e428...9b5fd67. Read the comment docs.

@wu-sheng wu-sheng merged commit 25e4f28 into apache:master Jun 25, 2022
@CherishCai CherishCai changed the title Feature(tonic-build): set tonic-build.build_server(false), do not build Server code. Chore(tonic-build): set tonic-build.build_server(false), do not build Server code. Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Housekeeping things like CI, license things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants