Brunda10 opened a new pull request, #54597:
URL: https://github.com/apache/airflow/pull/54597

   Closes: [#53901](https://github.com/apache/airflow/issues/53901)
   
   Most single resource PATCH APIs already support **update_mask**, but bulk 
APIs don’t. This makes the behavior inconsistent.
   
   In this PR, I’ve added update_mask at the action level for bulk updates. 
Since bulk APIs already work with action-based payloads (create/update/delete), 
keeping update_mask inside the action felt cleaner than adding it as a global 
query parameter.
   
   For single APIs I followed the same logic as before:
   
   If update_mask is passed → only those fields are updated.
   
   If not → all fields in the request body are updated.
   
   I haven’t made strict validation (rejecting when update_mask is missing), 
but I can update if that’s the expected behavior.
   
   Looking for feedback on whether action-level update_mask for bulk is the 
right approach or if we should make it global.


-- 
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

Reply via email to