[feat]: [CI-12466]: Initialize optimizationState to empty string #279
Conversation
There was a problem hiding this comment.
@anurag-harness why is this change required? The empty string will still be part of the JSON. Backward compatibility is taken care of by ignoreUnknown annotation.
If the delegate is very old, we don't guarantee backward compatibility anyway. Please check with delegate team the number of releases after which backward compatibility is not supported/guaranteed.
@rutvijmehta-harness Due to the omitempty in the structure, the field will be omitted if it is an empty string |
optimizationState should be initialized to an empty string and should only contain a non empty value if the FF to get optimization state is enabled. This will ensure backward compatibility of a vm runner with an older delegate