potiuk merged PR #41576:
URL: https://github.com/apache/airflow/pull/41576
--
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.a
moiseenkov commented on PR #41576:
URL: https://github.com/apache/airflow/pull/41576#issuecomment-2324601110
> Looks good overall!
> I got a small comment regarding reusability.
@shahar1 ,
Thanks for the review. Please take a look at the fixes.
--
This is an automated message
moiseenkov commented on code in PR #41576:
URL: https://github.com/apache/airflow/pull/41576#discussion_r1740803481
##
airflow/providers/google/cloud/operators/dataflow.py:
##
@@ -945,6 +948,11 @@ def on_kill(self) -> None:
)
+@deprecated(
+reason="DataflowS
moiseenkov commented on code in PR #41576:
URL: https://github.com/apache/airflow/pull/41576#discussion_r1740803070
##
airflow/providers/google/cloud/hooks/dataflow.py:
##
@@ -938,6 +937,110 @@ def launch_job_with_flex_template(
response: dict = request.execute(num_retr
shahar1 commented on code in PR #41576:
URL: https://github.com/apache/airflow/pull/41576#discussion_r1726678936
##
airflow/providers/google/cloud/hooks/dataflow.py:
##
@@ -938,6 +937,110 @@ def launch_job_with_flex_template(
response: dict = request.execute(num_retries
shahar1 commented on code in PR #41576:
URL: https://github.com/apache/airflow/pull/41576#discussion_r1726678936
##
airflow/providers/google/cloud/hooks/dataflow.py:
##
@@ -938,6 +937,110 @@ def launch_job_with_flex_template(
response: dict = request.execute(num_retries
moiseenkov opened a new pull request, #41576:
URL: https://github.com/apache/airflow/pull/41576
Add DataflowStartYamlJobOperator as a replacement option for
DataflowStartSqlJobOperator.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
