github-actions[bot] opened a new pull request, #75476:
URL: https://github.com/apache/airflow/pull/75476

   * Go SDK: fail a task whose return-value XCom push fails, fix the Label doc
   
   A failed return-value push was logged and the task was left successful, so 
the
   downstream tasks that read it failed later with XComNotFound. Fail the task
   instead, and stop pushing a result the task never produced when its function
   already returned an error, as the branch path does.
   
   The guide's labelled-edge example did not compile: Label takes only the text,
   and On names the tasks its edges reach.
   
   * Go SDK: address review on the return-value XCom push
   
   Push the task instance from the runtime context as is, share its lookup
   with startBranch, and cover the failed push on the branch path. Merge the
   copied test helper into runTask.
   
   * Go SDK: test that a task pushes no nil-pointer return value
   
   Only the branch path covered this check, and a nil pointer that slipped
   past it would be pushed as null now that a push can fail the task.
   
   ---------
   (cherry picked from commit 1a50a18a3c604e75c6f5f059e90e4a5e8777fcf0)
   
   Co-authored-by: Jason(Zhe-You) Liu 
<[email protected]>


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

Reply via email to