Re: [PR] AIP-72: Extending SET RTIF endpoint to accept all JSONable types [airflow]

[via GitHub]

amoghrajesh commented on code in PR #44843:
URL: https://github.com/apache/airflow/pull/44843#discussion_r1881450046


##########
task_sdk/src/airflow/sdk/execution_time/comms.py:
##########
@@ -170,13 +170,17 @@ class PutVariable(BaseModel):
     type: Literal["PutVariable"] = "PutVariable"
 
 
+"""Defines the types that the RTIF payload's dictionary values can take. These 
are all JsonAble types """
+JsonAbleValueTypes = Union[str, dict[str, str], list[str], int, float, None]

Review Comment:
   @kaxil you are right. I tried digging in to this deeply and on debugging, I 
can see tuple will be present for cases like `op_args`. However, I see thji
   
![image](https://github.com/user-attachments/assets/3e2cb006-7309-48c8-bd30-b439dea242cd)
   



-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org