Re: [apache/incubator-teaclave] Task logger (PR #676)

2023-02-14 Thread He Sun
Keeping the logs persistent is good for audit. The log is more a supplement to `TaskState` that is persistent in the `storage` service than just a `diagnosis` tool for developers here. Let me have a think and we can achieve both. -- Reply to this email directly or view it on GitHub: https://git

Re: [apache/incubator-teaclave] Task logger (PR #676)

2023-02-14 Thread Mingshen Sun
The original design is to separate executors with other components to make it more flexible. In this implementation, if I understand correctly, executors are talking to the storage directly. Why not return the logs with the function return value instead? -- Reply to this email directly or vie