1fanwang commented on issue #70953:
URL: https://github.com/apache/airflow/issues/70953#issuecomment-5186691501

   Verified [Validate template fields after rendering in common.ai 
operators](https://github.com/apache/airflow/pull/70338) on `common-ai 
0.7.0rc2`. Works as intended.
   
   Airflow 3.3.0, two identical virtualenvs: one with the RC providers, one 
with the previous stable release. Same Dag, same command, run through the real 
task runner. Only the external transport is stubbed, and the stub asserts the 
value that actually reaches it.
   
   A `DocumentLoaderOperator` where `source_path` renders to `None` and 
`source_bytes` carries the content, so exactly one source is set once rendering 
has happened.
   
   ```bash
   # common-ai 0.7.0rc2
   AIRFLOW_HOME=$PWD/airflow_home \
   AIRFLOW__CORE__DAGS_FOLDER=$PWD/dags_e2e/rc_70338_common_ai \
     .venv/bin/airflow dags test rc_70338_common_ai
   
   # common-ai 0.6.0, identical Dag
   AIRFLOW_HOME=$PWD/airflow_home_prev \
   AIRFLOW__CORE__DAGS_FOLDER=$PWD/dags_e2e/rc_70338_common_ai \
     .venv-prev/bin/airflow dags test rc_70338_common_ai
   ```
   
   ```
   # common-ai 0.7.0rc2
   RC_CHECK_OK document parsed from source_bytes after source_path rendered to 
None
   XCom: [{'text': 'hello from rc', 'metadata': {}}]
   state=success
   
   # common-ai 0.6.0, no task ever runs
   Error loading DAG from rc_70338_common_ai.py:
   ValueError: Provide exactly one of 'source_path' or 'source_bytes', not both.
   ```


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