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

Feature/grpc #451

Merged
merged 85 commits into from
Sep 13, 2018
Merged

Feature/grpc #451

merged 85 commits into from
Sep 13, 2018

Conversation

namtruong
Copy link
Collaborator

Add gRPC capability for tessera. The communication protocol can be switchable using an additional config flag communicationType (possible values REST, GRPC). Default is REST

namtruong and others added 30 commits August 22, 2018 14:24
that mediates between enclave and base64 codecs.Rename grpc port 
to follow java conventions. Add convertor that copies properties 
to and from grpc objects and the existing rest model objects. 

Added initial acceptance test for Grpc send.
Port logic from grpc and rest resources into delegate object
when do config migration default this to be using REST
Fix unused imports in app module. 
Only run rest tests from failsafe for now.
into specialised java object rather than doing this in groovy.
@codecov-io
Copy link

codecov-io commented Sep 11, 2018

Codecov Report

Merging #451 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #451      +/-   ##
===========================================
- Coverage      99.7%   99.7%   -0.01%     
- Complexity     1033    1106      +73     
===========================================
  Files           163     176      +13     
  Lines          3440    3669     +229     
  Branches        189     189              
===========================================
+ Hits           3430    3658     +228     
- Partials         10      11       +1
Impacted Files Coverage Δ Complexity Δ
...orum/tessera/util/exception/DecodingException.java 100% <ø> (ø) 1 <0> (?)
.../java/com/quorum/tessera/node/model/PartyInfo.java 100% <ø> (ø) 4 <0> (?)
.../main/java/com/quorum/tessera/api/ApiResource.java 100% <ø> (ø) 5 <0> (?)
.../tessera/api/exception/DefaultExceptionMapper.java 100% <ø> (ø) 3 <0> (?)
...n/java/com/quorum/tessera/client/PostDelegate.java 100% <ø> (ø) 5 <0> (?)
...n/java/com/quorum/tessera/api/VersionResource.java 100% <ø> (ø) 3 <0> (?)
...a/api/exception/WebApplicationExceptionMapper.java 100% <ø> (ø) 4 <0> (?)
...com/quorum/tessera/exception/TesseraException.java 100% <ø> (ø) 2 <0> (?)
...a/com/quorum/tessera/api/model/ReceiveRequest.java 100% <ø> (ø) 5 <0> (?)
...main/java/com/quorum/tessera/node/model/Party.java 100% <ø> (ø) 6 <0> (?)
... and 63 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 318ee6b...1330609. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.7 enhancement Non-user code enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants