ga4gh / task-execution-schemas

Apache License 2.0
80 stars 28 forks source link

Clarify Executor.stdout/err docs and implementation requirements. #94

Closed buchanae closed 6 years ago

buchanae commented 6 years ago

While developing Funnel, we've found that ensuring that these logs are reliably the correct tail of the content is tricky and feels like an unnecessary burden to implementors. I'm proposing we remove the requirement of the tail, and free up implementations to do what works best for them and their users. There is a fallback of writing the stdout/err to a file and uploading that file to permanent storage, as a way to consistently guarantee you get the full content.

pditommaso commented 6 years ago

👍