Skip to content

refactor: break down server initialisation in different parts #240

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.

Sign up for GitHub

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

Jump to bottom
Merged
sbchaos merged 26 commits into from
Mar 28, 2022
Merged

refactor: break down server initialisation in different parts #240

sbchaos merged 26 commits into from
Mar 28, 2022

Conversation

@sbchaos
Copy link
Contributor

@sbchaos sbchaos commented Mar 24, 2022

No description provided.

sbchaos added 17 commits March 24, 2022 10:11
@sbchaos
refactor: move server dependencies to separate file
7a0e885
@sbchaos
refactor: move init of db to separate function
ba7566b
@sbchaos
refactor: move code for appkey up
35f3b2b
@sbchaos
refactor: move some dependencies up
7d3fc0c
@sbchaos
refactor: extract function to init schedulers
c6461da
@sbchaos
refactor: extract function to setup grpc server
11a798a
@sbchaos
refactor: move grpc-address out of setup-grpc-server
2059f83
@sbchaos
refactor: extract http proxy function
ce0bd3b
@sbchaos
refactor: extract function to init prime cluster
565bf0e
@sbchaos
feat: add optimus server
211ef0d
@sbchaos
refactor: add methods to init various systems
9df81be
@sbchaos
refactor: rearrange the order of functions
fdaf703
@sbchaos
refactor: simplify init of the scheduler
35e3acb
@sbchaos
refactor: use optimus server in serve cmd
196084f
@sbchaos
refactor: remove unused and annotate code
b6b3ead
@sbchaos
refactor: extract scheduler initialisation to separate file
6f06c8e
@sbchaos
refactor: cleanup for messages and db init
9b79891
@coveralls
Copy link

coveralls commented Mar 24, 2022 •
edited
Loading

Pull Request Test Coverage Report for Build 2052397302

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 74.259%
Totals Coverage Status
Change from base Build 2051761049: 0.0%
Covered Lines: 5487
Relevant Lines: 7389
💛 - Coveralls

@sbchaos sbchaos linked an issue Mar 24, 2022 that may be closed by this pull request
Split and Simplify the server inititialization in optimus/cmd/server/server.go #167
Closed
@deryrahman deryrahman mentioned this pull request Mar 25, 2022
Merged
5 tasks
sravankorumilli
sravankorumilli approved these changes Mar 28, 2022
View reviewed changes
@sbchaos sbchaos merged commit 21b96c2 into main Mar 28, 2022
@sbchaos sbchaos deleted the refactor-server branch March 28, 2022 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sravankorumilli sravankorumilli sravankorumilli approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Split and Simplify the server inititialization in optimus/cmd/server/server.go

4 participants

@sbchaos @coveralls @sravankorumilli