You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by Zhaofeng Chen <no...@github.com> on 2020/02/05 05:26:43 UTC
[apache/incubator-teaclave] [types] Remove ser/de for worker structs
with polished tests (#217)
## Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
## Type of change (select applied and DELETE the others)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] API change with a documentation update
- [ ] Additional test coverage
- [ ] Code cleanup or just sync with upstream third-party crates
## How Has This Been Tested?
## Checklist (check ALL before submitting PR, even not applicable)
- [ ] Fork the repo and create your branch from `master`.
- [ ] If you've added code that should be tested, add tests.
- [ ] If you've changed APIs, update the documentation.
- [ ] Ensure the tests pass (see CI results).
- [ ] Make sure your code lints/format.
You can view, comment on, or merge this pull request online at:
https://github.com/apache/incubator-teaclave/pull/217
-- Commit Summary --
* [types] Remove ser/de for worker structs with polished tests
-- File Changes --
M services/proto/src/teaclave_common.rs (51)
M services/proto/src/teaclave_execution_service.rs (11)
M tests/functional_tests/enclave/src/teaclave_execution_service.rs (103)
M tests/integration_tests/enclave/src/teaclave_worker.rs (121)
M types/Cargo.toml (1)
M types/src/crypto.rs (145)
M types/src/worker.rs (101)
M worker/src/function/context.rs (2)
M worker/src/function/gbdt_training.rs (95)
M worker/src/function/mesapy.rs (29)
-- Patch Links --
https://github.com/apache/incubator-teaclave/pull/217.patch
https://github.com/apache/incubator-teaclave/pull/217.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/217
Re: [apache/incubator-teaclave] [types] Remove ser/de for worker
structs with polished tests (#217)
Posted by Mingshen Sun <no...@github.com>.
Merged #217 into develop.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-teaclave/pull/217#event-3008740826