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.

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

refactor: upgrade implementation of GRPC (from grpcio to tonic) #18

Merged
merged 12 commits into from
Nov 10, 2022
Merged

refactor: upgrade implementation of GRPC (from grpcio to tonic) #18

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e5f990a
refactor: replace the grpcio crate with tonic
archerny Nov 9, 2022
8e7fef8
refactor: fmt code with
archerny Nov 9, 2022
b787c66
refactor: refact some code according to clippy's advice
archerny Nov 9, 2022
2c64f6e
refactor: add some modifications
archerny Nov 10, 2022
fa92fcd
refactor: modify CI workflow in order to support tonic
archerny Nov 10, 2022
e402a97
fix: fix some issues that commes from clippy and CR
archerny Nov 10, 2022
ec08ba0
refactor: adopt suggestion for src/model/write/request.rs
archerny Nov 10, 2022
b16ec57
refactor: adopt suggestion for src/rpc_client/rpc_client_impl.rs
archerny Nov 10, 2022
9c975e7
refactor: adopt some CR advice, and make some improvements
archerny Nov 10, 2022
d7eb846
refactor: transmit more detailed auth error
archerny Nov 10, 2022
9ca2363
refactor: remove unused import
archerny Nov 10, 2022
cab2518
refactor: fix code format issue
archerny Nov 10, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ jobs:
- name: Setup Build Environment
run: |
apt update
apt install --yes gcc g++ libssl-dev pkg-config cmake
apt install --yes gcc g++ libssl-dev pkg-config cmake protobuf-compiler
rm -rf /var/lib/apt/lists/*
- name: Install Clippy
run: |
Expand Down Expand Up @@ -122,7 +122,7 @@ jobs:
- name: Setup Build Environment
run: |
apt update
apt install --yes gcc g++ libssl-dev pkg-config cmake
apt install --yes gcc g++ libssl-dev pkg-config cmake protobuf-compiler
rm -rf /var/lib/apt/lists/*
- name: Run Test
run: |
Expand Down
Loading