uranusjr opened a new pull request, #76007: URL: https://github.com/apache/airflow/pull/76007
Identified while reviewing #75920. A successful Variable read never carries a null value. `Variable.get` raises `KeyError` when nothing is found, and the route turns that into a 404. The Execution API and supervisor schema still typed the value as `str | None`, so every generated client got a nullable type. This change the variable value’s type to non-nullable, so null only means "not found". The Task SDK allowed `Variable.set(key, None)`, but internally the *None* would be stringified into `"None"` before being sent through the wire. I also deprecated this call since it is too confusing IMO. The task and asset state store read endpoints return 404 for a stored JSON null, so a successful read never carries null. Every writer already rejects null, so the spec stays `JsonValue`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
