Store/represent time offsets as strings
instead of int + negative_utc boolean
The plan:
- Add
offset_bytes
attribute toTimestampWithTimezone
, fill it fromoffset
andnegative_utc
if not provided - Make
offset
andnegative_utc
optional (fill the attributes fromoffset_bytes
) - Raise DeprecationWarning when they are passed to the constructor (but
from_dict
should not raise it) - Remove explicit uses in the constructor (track down deprecation warnings)
- Remove use of
negative_utc
(there is one inswh-web/swh/web/common/converters.py
) - Remove them from the constructor (but not
from_dict
, because kafka), removeoffset
andnegative_utc
entirely Renameoffset_bytes
tooffset
Postgresql must be migrated somewhere between 1 and 6 (exclusive). Kafka does not need to be migratet as long as we keep the compat in from_dict
methods.
Migrated from T3752 (view on Phabricator)
Edited by Phabricator Migration user