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&#39;ve added code that should be tested, add tests.
- [ ] If you&#39;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