You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by yc-huang <no...@github.com> on 2020/07/17 09:18:27 UTC

[apache/incubator-teaclave] builtin_echo failed with large input (#392)

The python client will fail with large input arguments; this could be replicated by running builtin_echo example with large input (e.g. the project LICENSE file, size 11887).

-  increase the heap size of both storage and management service (service/[storage, management]/enclave/Enclave.config.xml) to 0x10000000 could avoid the oom issue;
-  it seems that the rpc reqs/resps could be too verbose / low efficiency;
-  it seems that the response payload was truncated somewhere to 16k(16384) and get malformed.



-- 
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/issues/392

Re: [apache/incubator-teaclave] builtin_echo failed with large input (#392)

Posted by Mingshen Sun <no...@github.com>.
I have increased the default to 256M in this commit #396 and fixed the truncated response issue in the Python client SDK.

For the large input, we don't suggest to send as function parameters and we may consider to limit the size of an RPC message later. One reason is that the current RPC is in JSON format which does not support raw binary data very well. That's why large inputs consume many memory when parsing the RPC messages. This situation will be improved if we support HTTP(s).

-- 
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/issues/392#issuecomment-666741079

Re: [apache/incubator-teaclave] builtin_echo failed with large input (#392)

Posted by yc-huang <no...@github.com>.
Thanks @mssun,  my local test confirmed that this had been fixed.

-- 
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/issues/392#issuecomment-666998729

Re: [apache/incubator-teaclave] builtin_echo failed with large input (#392)

Posted by yc-huang <no...@github.com>.
Closed #392.

-- 
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/issues/392#event-3607428779