You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@teaclave.apache.org by Mingshen Sun <no...@github.com> on 2020/05/21 01:21:08 UTC

[apache/incubator-teaclave] Use json type for function arguments instead of string/string hashmap (#307)

## Description

Use json type (`serde_json::Value`) for function arguments instead of string hashmap (`HashMap&lt;String, String&gt;`).

CI passed: https://ci.mesalock-linux.org/mssun/incubator-mesatee/799
You can view, comment on, or merge this pull request online at:

  https://github.com/apache/incubator-teaclave/pull/307

-- Commit Summary --

  * Use json type for function arguments instead of string/string hashmap

-- File Changes --

    M function/src/echo.rs (13)
    M function/src/gbdt_train.rs (48)
    M function/src/logistic_regression_train.rs (24)
    M sdk/python/teaclave.py (1)
    M services/execution/enclave/src/service.rs (28)
    M services/management/enclave/src/service.rs (3)
    M services/proto/src/proto/teaclave_frontend_service.proto (4)
    M services/proto/src/teaclave_frontend_service.rs (8)
    M tests/functional/enclave/src/end_to_end/builtin_gbdt_train.rs (24)
    M tests/integration/enclave/src/teaclave_worker.rs (24)
    M types/src/staged_function.rs (129)

-- Patch Links --

https://github.com/apache/incubator-teaclave/pull/307.patch
https://github.com/apache/incubator-teaclave/pull/307.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/307

Re: [apache/incubator-teaclave] Use json type for function arguments instead of string/string hashmap (#307)

Posted by Mingshen Sun <no...@github.com>.

Merged #307 into master.

-- 
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/307#event-3359046679

Re: [apache/incubator-teaclave] Use json type for function arguments instead of string/string hashmap (#307)

Posted by Mingshen Sun <no...@github.com>.

Please approve again. Thanks!

-- 
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/307#issuecomment-631824743