kaxil commented on code in PR #51699:
URL: https://github.com/apache/airflow/pull/51699#discussion_r2150572111
##########
airflow-core/src/airflow/dag_processing/processor.py:
##########
@@ -102,18 +101,16 @@ class DagFileParsingResult(BaseModel):
def _parse_file_entrypoint():
import structlog
- from airflow.sdk.execution_time import task_runner
+ from airflow.sdk.execution_time import comms, task_runner
# Parse DAG file, send JSON back up!
- comms_decoder = task_runner.CommsDecoder[ToDagProcessor, ToManager](
- input=sys.stdin,
- decoder=TypeAdapter[ToDagProcessor](ToDagProcessor),
+ comms_decoder = comms.CommsDecoder[ToDagProcessor, ToManager](
+ body_decoder=TypeAdapter[ToDagProcessor](ToDagProcessor),
)
- msg = comms_decoder.get_message()
+ msg = comms_decoder._get_response()
Review Comment:
Worth adding a code comment, so it doesn't look weird here on why we access
an internal method
--
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]
