Skip to content

ppc: Rename to pps and refactor#398

Merged
bors[bot] merged 1 commit intomasterfrom
pps
Aug 26, 2020
Merged

ppc: Rename to pps and refactor#398
bors[bot] merged 1 commit intomasterfrom
pps

Conversation

@MikailBag
Copy link
Copy Markdown
Member

@MikailBag MikailBag commented Aug 21, 2020

old name: problem package compiling (not cool)
new name: problem preparation system (or services, as you wish) (cooler)

new architecture: CLI and server

CLI starts server in background and sends RPC requests to the server.
Server actually does stuff.

In future it will be possible to start one server which will process
requests from many clients.

RPC protocol is based on HTTP POST requests and defined via Rust
structs. In future, we should make consuming PPS api from other
languages easier (e.g. provide JSON schema for types).

Also, it's worth extracting RPC implementation into jjs-commons. (UPD: done)

@MikailBag
Copy link
Copy Markdown
Member Author

Also, in future pps will live in other repository in jjs-dev org.

@MikailBag MikailBag force-pushed the pps branch 3 times, most recently from 063edbd to 9dcf426 Compare August 22, 2020 15:27
@MikailBag MikailBag marked this pull request as ready for review August 24, 2020 10:42
Comment thread src/invoker/src/lib.rs
@@ -1,4 +1,4 @@
#![feature(proc_macro_hygiene, type_alias_impl_trait)]
#![type_length_limit = "4323264"]
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is issue in either rust or tracing.

old name: problem package compiling (not cool)
new name: problem preparation system (or services, as you wish) (cooler)

new architecture: CLI and server

CLI starts server in background and sends RPC requests to the server.
Server actually does stuff.

In future it will be possible to start one server which will process
requests from many clients.

RPC protocol is based on HTTP POST requests and defined via Rust
structs. In future, we should make consuming PPS api from other
languages easier (e.g. provide JSON schema for types).

Also, it's worth extracting RPC implementation into jjs-commons.
@MikailBag
Copy link
Copy Markdown
Member Author

bors r+

@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Aug 26, 2020

Build succeeded:

@bors bors Bot merged commit aced5b1 into master Aug 26, 2020
@bors bors Bot deleted the pps branch August 26, 2020 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant