girder / girder_worker

Distributed task execution engine with Girder integration, developed by Kitware
http://girder-worker.readthedocs.io/
Apache License 2.0
34 stars 30 forks source link

Only send log value if it has data #364

Closed zachmullen closed 4 years ago

zachmullen commented 4 years ago

Girder's server-side handler writes data to the job log even if it's just an empty string, which ends up writing e.g. ["", "", "", ""] to the database, growing linearly with each progress update event. It's debatable whether that server side behavior is desirable, however it's easy enough to handle it here and only send the log value if something has been written to it.

zachmullen commented 4 years ago

Tagged and pushed this on v0.7.2.