For deserialization safety, can you add something here like: - if "status" is null and "statusCode" is not null, set both to the value of "statusCode" - If "statusCode" is null and "status" is not null, set both to the value of "status" - if both are non-null, but have different values, log an error
I don't think there should be a situation where new code would be trying to deserialize an old TaskStatusPlus if the rolling upgrade sequence is followed, but I think it'd be good to handle that situation [ Full content available at: https://github.com/apache/incubator-druid/pull/6334 ] This message was relayed via gitbox.apache.org for [email protected]
