Skip to content

Use None instead of default initialized gRPC objects

Rohit Kothur requested to merge rkothur/fix-job-timestamps into master

Description

When I submitted !210 (diffs) I should have set the default parameters to None rather than a Timestamp.

Currently, we always CopyFrom, meaning the timestamps will use the default-constructed Timestamp if it's not passed into the constructor, which will likely be very old.

Edited by Rohit Kothur

Merge request reports