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

2023-02-07 Thread He Sun
## Description Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. Fixes # (issue) https://github.com/apache/incubator-teaclave/issues/669 The task logger part. Platform logging will be in another PR in a while. ## Type of chan

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

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-03-06 Thread He Sun
Merged #676 into master. -- Reply to this email directly or view it on GitHub: https://github.com/apache/incubator-teaclave/pull/676#event-8681744546 You are receiving this because you are subscribed to this thread. Message ID: