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


##########
task-sdk/src/airflow/sdk/bases/resumablejobmixin.py:
##########
@@ -101,29 +107,56 @@ def execute_resumable(self, context: Context) -> Any:
         Closing this window would require atomic "submit + persist", which is 
not possible across
         an external system boundary.
         """
-        task_store = context.get("task_store")
+        operator_tag = {"operator": type(self).__name__}
+
+        with tracer.start_as_current_span("resumable_job.resume_decision") as 
span:

Review Comment:
   Sorry, dont follow, can you share more context on what you are seeing? 
   
   The span is created exactly once per `execute_resumable()` call, so I expect 
it to appear only once in the trace. Where are you seeing two distinct spans 
with different timestamps/durations, or is it the same span being rendered 
under two parents in jaeger tree view? 



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