uranusjr commented on code in PR #33570:
URL: https://github.com/apache/airflow/pull/33570#discussion_r1309928233
##
airflow/models/dag.py:
##
@@ -717,6 +718,11 @@ def validate_setup_teardown(self):
:meta private:
"""
for task in self.tasks:
+
bfeif opened a new issue, #33911:
URL: https://github.com/apache/airflow/issues/33911
### Description
This is mostly self-explanatory. The `BigQueryHook` already has a
`get_pandas_df()`; this issue aims to add an analogous `get_polars_df()` method
that would return a polars
boring-cyborg[bot] commented on issue #33911:
URL: https://github.com/apache/airflow/issues/33911#issuecomment-1698795744
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for
Lee-W commented on PR #33825:
URL: https://github.com/apache/airflow/pull/33825#issuecomment-1698795945
> Right - this is a "Real" problem now you see.
Got it. Just get it fixed. Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please
potiuk commented on PR #857:
URL: https://github.com/apache/airflow-site/pull/857#issuecomment-1699083119
I think it's best to link to one of your GitHub repos and have a readme
there explaining how to integrate airflow.
--
This is an automated message from the Apache Git Service.
To
derbeneviv commented on issue #33894:
URL: https://github.com/apache/airflow/issues/33894#issuecomment-1699092379
[Taragolis](https://github.com/Taragolis)
well, i didn't, and it seems to work
```
{"endpoint_url": "https://mu_url;, "verify": false}
```
however, that's
wolfdn commented on PR #33914:
URL: https://github.com/apache/airflow/pull/33914#issuecomment-1699123215
> Does this PR aim to solve #33692 ?
I think https://github.com/apache/airflow/issues/33692 is unrelated to the
issue that I'm trying to solve here.
This PR is about the
eladkal closed issue #9005: Hive Operator mapred_job_name_template missing from
config template
URL: https://github.com/apache/airflow/issues/9005
--
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
eladkal commented on issue #9005:
URL: https://github.com/apache/airflow/issues/9005#issuecomment-1698715407
Closing as stale issue
--
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
potiuk commented on code in PR #33637:
URL: https://github.com/apache/airflow/pull/33637#discussion_r1309882203
##
setup.cfg:
##
@@ -22,7 +22,7 @@ author = Apache Software Foundation
author_email = d...@airflow.apache.org
url = https://airflow.apache.org/
version = attr:
uranusjr commented on code in PR #33570:
URL: https://github.com/apache/airflow/pull/33570#discussion_r1309949531
##
airflow/ti_deps/deps/trigger_rule_dep.py:
##
@@ -121,6 +132,13 @@ def _evaluate_trigger_rule(
from airflow.models.operator import needs_expansion
Taragolis commented on issue #33688:
URL: https://github.com/apache/airflow/issues/33688#issuecomment-1698823134
> here is the performance activity for the last 5 hours. The peak at 3 AM we
see represents also a peak in activity in terms of DAGs running. Anything
particular you want to
uranusjr commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1698991410
Is `TaskInstanceState.SHUTDOWN` documented, or just `State.SHUTDOWN`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
avkirilishin opened a new pull request, #33915:
URL: https://github.com/apache/airflow/pull/33915
Fix the issue #33164
@ephraimbuddy Do you remember, why `if ti.map_index > -1:` was added?
---
**^ Add meaningful description above**
Read the
pankajastro commented on issue #33875:
URL: https://github.com/apache/airflow/issues/33875#issuecomment-1699134019
https://github.com/apache/airflow/pull/33627,
https://github.com/apache/airflow/pull/33598 and
https://github.com/apache/airflow/pull/33696 looks good
--
This is an
potiuk commented on code in PR #33637:
URL: https://github.com/apache/airflow/pull/33637#discussion_r1309882203
##
setup.cfg:
##
@@ -22,7 +22,7 @@ author = Apache Software Foundation
author_email = d...@airflow.apache.org
url = https://airflow.apache.org/
version = attr:
khmelevskiy opened a new issue, #33912:
URL: https://github.com/apache/airflow/issues/33912
### Apache Airflow version
2.7.0
### What happened
If someone rename dag, than count became +1.
At the moment I have two times less dags, than I see in web ui.
Past
RamanDamayeu commented on PR #857:
URL: https://github.com/apache/airflow-site/pull/857#issuecomment-1698886905
Thanks, very valid point!
Of course, I left a link to the implementation of ODD integration with
Airflow in a comment, but I agree that it will not be very easy for users
wolfdn opened a new pull request, #33914:
URL: https://github.com/apache/airflow/pull/33914
We are using the DockerOperator and noticed that the logs are a bit
confusing when there is an error in a Docker container (return code != 0)
because Airflow shows the log output of the Docker
boring-cyborg[bot] commented on PR #33914:
URL: https://github.com/apache/airflow/pull/33914#issuecomment-1698955427
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our Contribution
ashb commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1698956131
We need to add `TaskInstanceState.SHUTDOWN` back even if nothing uses it --
this counts as a breaking change since it has been documented and exists in the
code.
By removing this
uranusjr commented on code in PR #33570:
URL: https://github.com/apache/airflow/pull/33570#discussion_r1309953671
##
airflow/ti_deps/deps/trigger_rule_dep.py:
##
@@ -234,59 +296,72 @@ def _iter_upstream_conditions() ->
Iterator[ColumnOperators]:
uranusjr commented on code in PR #33864:
URL: https://github.com/apache/airflow/pull/33864#discussion_r1309970337
##
tests/providers/google/cloud/hooks/test_mlengine.py:
##
@@ -1009,11 +1009,11 @@ def test_delete_version(self, mock_get_conn,
mock_project_id):
uranusjr commented on code in PR #33720:
URL: https://github.com/apache/airflow/pull/33720#discussion_r1310023089
##
airflow/models/taskreschedule.py:
##
@@ -142,9 +154,93 @@ def find_for_task_instance(
:param try_number: Look for TaskReschedule of the given
eladkal merged PR #33712:
URL: https://github.com/apache/airflow/pull/33712
--
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:
eladkal closed issue #33711: EcsRunTaskOperator waiter default
waiter_max_attempts too low - all Airflow tasks detach from ECS tasks at 10
minutes
URL: https://github.com/apache/airflow/issues/33711
--
This is an automated message from the Apache Git Service.
To respond to the message,
boring-cyborg[bot] commented on PR #33712:
URL: https://github.com/apache/airflow/pull/33712#issuecomment-1698929785
Awesome work, congrats on your first merged pull request! You are invited to
check our [Issue Tracker](https://github.com/apache/airflow/issues) for
additional
eladkal opened a new issue, #33916:
URL: https://github.com/apache/airflow/issues/33916
### Body
Now that we have Executors in providers we should migrate
https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/executor/celery.html
mzecov opened a new issue, #33918:
URL: https://github.com/apache/airflow/issues/33918
### Apache Airflow version
2.7.0
### What happened
Variable `{{ prev_start_date_success }}` returns unexpected results when
manually triggering DAG.
With following DAG
boring-cyborg[bot] commented on issue #33918:
URL: https://github.com/apache/airflow/issues/33918#issuecomment-1699117236
Thanks for opening your first issue here! Be sure to follow the issue
template! If you are willing to raise PR to address this issue please do so, no
need to wait for
uranusjr commented on PR #33900:
URL: https://github.com/apache/airflow/pull/33900#issuecomment-1698728456
Can a test be added for this?
--
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
pankajkoti commented on code in PR #33637:
URL: https://github.com/apache/airflow/pull/33637#discussion_r1309958647
##
setup.cfg:
##
@@ -22,7 +22,7 @@ author = Apache Software Foundation
author_email = d...@airflow.apache.org
url = https://airflow.apache.org/
version = attr:
uranusjr commented on code in PR #33720:
URL: https://github.com/apache/airflow/pull/33720#discussion_r1309968634
##
airflow/models/taskreschedule.py:
##
@@ -142,9 +154,93 @@ def find_for_task_instance(
:param try_number: Look for TaskReschedule of the given
Taragolis commented on code in PR #33720:
URL: https://github.com/apache/airflow/pull/33720#discussion_r1310007883
##
airflow/models/taskreschedule.py:
##
@@ -142,9 +154,93 @@ def find_for_task_instance(
:param try_number: Look for TaskReschedule of the given
alephmelo commented on issue #31871:
URL: https://github.com/apache/airflow/issues/31871#issuecomment-1698983506
@eladkal Would you be able to review the PR I opened a few days ago?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk opened a new pull request, #33920:
URL: https://github.com/apache/airflow/pull/33920
Small addition to our security model - it was not entirely clear that DAG
authors can also execute code in DAG File Processor and that DAG File Processor
can be run in standalone mode effectively
pankajastro merged PR #33822:
URL: https://github.com/apache/airflow/pull/33822
--
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:
pankajastro merged PR #33825:
URL: https://github.com/apache/airflow/pull/33825
--
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:
ephraimbuddy commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1698897191
Looks like we should add a significant news fragment for this @Bisk1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
uranusjr commented on PR #33915:
URL: https://github.com/apache/airflow/pull/33915#issuecomment-1699121844
The `map_index > -1` condition ensures this only handles mapped task, since
e91637f8894cac19c6b467b6669cbcc13184be70 intends to only fix a deadlock that
affects mapped tasks. So the
pankajkoti commented on PR #33637:
URL: https://github.com/apache/airflow/pull/33637#issuecomment-1699142821
Static check is failing with the below error:
```
Check if licenses are OK for
ApacheFailed
- hook id:
Lee-W commented on PR #33822:
URL: https://github.com/apache/airflow/pull/33822#issuecomment-1698797957
Thanks! I couldn't reproduce it locally yesterday and thus feel a bit
confused. I'll take a deeper look next time.
--
This is an automated message from the Apache Git Service.
To
mobuchowski commented on issue #33909:
URL: https://github.com/apache/airflow/issues/33909#issuecomment-1698815790
I like providers being standalone packages by default. In addition to the
benefits described here, it would allow installing providers straight from
github via pip.
I
Taragolis commented on code in PR #33720:
URL: https://github.com/apache/airflow/pull/33720#discussion_r1310041422
##
airflow/models/taskreschedule.py:
##
@@ -142,9 +154,93 @@ def find_for_task_instance(
:param try_number: Look for TaskReschedule of the given
eladkal commented on issue #33911:
URL: https://github.com/apache/airflow/issues/33911#issuecomment-1698925741
It needs to be added to `DbApiHook`
Adding function similar to:
uranusjr commented on code in PR #33637:
URL: https://github.com/apache/airflow/pull/33637#discussion_r1309878809
##
setup.cfg:
##
@@ -22,7 +22,7 @@ author = Apache Software Foundation
author_email = d...@airflow.apache.org
url = https://airflow.apache.org/
version = attr:
uranusjr commented on code in PR #33898:
URL: https://github.com/apache/airflow/pull/33898#discussion_r1310006395
##
airflow/providers/apache/spark/hooks/spark_submit.py:
##
@@ -461,31 +461,31 @@ def _process_spark_submit_log(self, itr: Iterator[Any])
-> None:
#
uranusjr commented on code in PR #33898:
URL: https://github.com/apache/airflow/pull/33898#discussion_r1310006395
##
airflow/providers/apache/spark/hooks/spark_submit.py:
##
@@ -461,31 +461,31 @@ def _process_spark_submit_log(self, itr: Iterator[Any])
-> None:
#
uranusjr commented on code in PR #33858:
URL: https://github.com/apache/airflow/pull/33858#discussion_r1309971825
##
airflow/www/views.py:
##
@@ -3796,11 +3796,11 @@ def historical_metrics_data(self):
data = {
"dag_run_types": {
uranusjr commented on code in PR #33898:
URL: https://github.com/apache/airflow/pull/33898#discussion_r1310004191
##
airflow/providers/amazon/aws/utils/__init__.py:
##
@@ -66,8 +66,8 @@ def datetime_to_epoch_us(date_time: datetime) -> int:
def get_airflow_version() ->
potiuk commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1698910499
> Looks like we should add a significant news fragment for this @Bisk1
@ephraimbuddy -> I am not so sure. Look at the discussion . It's really a
cleanup. SHUTDOWN had not been
fdemiane commented on issue #33875:
URL: https://github.com/apache/airflow/issues/33875#issuecomment-1699068069
Verified #33500
--
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
Taragolis closed issue #33894: AWS connection doesn't convert verify option to
bool
URL: https://github.com/apache/airflow/issues/33894
--
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
barrysteyn commented on issue #34093:
URL: https://github.com/apache/airflow/issues/34093#issuecomment-1706906717
@potiuk Thanks for the reply. I have just one final question (mainly because
I am new to the AirFlow community): How does it work from here? Would you like
me to submit a PR
jedcunningham merged PR #34106:
URL: https://github.com/apache/airflow/pull/34106
--
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:
potiuk merged PR #34101:
URL: https://github.com/apache/airflow/pull/34101
--
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:
vincbeck commented on code in PR #34051:
URL: https://github.com/apache/airflow/pull/34051#discussion_r1316043015
##
airflow/models/log.py:
##
@@ -37,6 +37,7 @@ class Log(Base):
event = Column(String(30))
execution_date = Column(UtcDateTime)
owner =
adam133 commented on code in PR #32903:
URL: https://github.com/apache/airflow/pull/32903#discussion_r1315921657
##
airflow/providers/databricks/hooks/databricks.py:
##
@@ -215,6 +216,67 @@ def find_job_id_by_name(self, job_name: str) -> int | None:
else:
potiuk merged PR #33309:
URL: https://github.com/apache/airflow/pull/33309
--
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:
potiuk merged PR #34102:
URL: https://github.com/apache/airflow/pull/34102
--
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:
potiuk merged PR #34097:
URL: https://github.com/apache/airflow/pull/34097
--
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:
ephraimbuddy merged PR #34027:
URL: https://github.com/apache/airflow/pull/34027
--
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:
ephraimbuddy closed issue #33949: Manual DAG triggers with Logical Date in the
Past trigger a second run when schedule is timedelta
URL: https://github.com/apache/airflow/issues/33949
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
josh-fell commented on code in PR #34106:
URL: https://github.com/apache/airflow/pull/34106#discussion_r1315995816
##
tests/providers/cncf/kubernetes/sensors/test_spark_kubernetes.py:
##
@@ -552,13 +552,6 @@
class TestSparkKubernetesSensor:
def setup_method(self):
potiuk merged PR #34104:
URL: https://github.com/apache/airflow/pull/34104
--
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:
barrysteyn commented on issue #34093:
URL: https://github.com/apache/airflow/issues/34093#issuecomment-1706773954
Thanks @potiuk. The ultimate aim of `allowed_deserialization_classes` is
provide protection against unexpected class deserialization. Using a regex is
goes against that - as
potiuk merged PR #34103:
URL: https://github.com/apache/airflow/pull/34103
--
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:
potiuk commented on PR #33959:
URL: https://github.com/apache/airflow/pull/33959#issuecomment-1706754244
rebased. seems GA are back
--
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
vincbeck commented on code in PR #33901:
URL: https://github.com/apache/airflow/pull/33901#discussion_r1316009816
##
airflow/auth/managers/base_auth_manager.py:
##
@@ -24,9 +24,11 @@
from airflow.utils.log.logging_mixin import LoggingMixin
if TYPE_CHECKING:
+from flask
potiuk merged PR #34096:
URL: https://github.com/apache/airflow/pull/34096
--
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:
potiuk commented on issue #34093:
URL: https://github.com/apache/airflow/issues/34093#issuecomment-1706899136
I am also fine for that. Regexp fallback was only there as a compatibility
measure and I am perfectly fine to do "glob-only" - that would by my way of
implemeting it if I were to
potiuk merged PR #34095:
URL: https://github.com/apache/airflow/pull/34095
--
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:
argibbs commented on issue #33410:
URL: https://github.com/apache/airflow/issues/33410#issuecomment-1706903000
Hello, sorry, not deliberately ignoring people, just not checking my mail
as often as I should.
(Insert real-life-getting-in-the-way comment here).
Haven't read the
Taragolis commented on issue #34105:
URL: https://github.com/apache/airflow/issues/34105#issuecomment-1706721682
Airflow 2.2.2 released almost two years ago, so that it quite a big chance
that it already resolved in the current Airflow:
potiuk merged PR #34091:
URL: https://github.com/apache/airflow/pull/34091
--
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:
Lee-W commented on code in PR #34104:
URL: https://github.com/apache/airflow/pull/34104#discussion_r1315763591
##
docs/apache-airflow-providers-microsoft-azure/connections/azure_container_volume.rst:
##
@@ -61,9 +63,9 @@ Extra (optional)
Specify the extra parameters (as
mhenc commented on code in PR #34026:
URL: https://github.com/apache/airflow/pull/34026#discussion_r1316039440
##
airflow/jobs/base_job_runner.py:
##
@@ -27,7 +27,7 @@
from airflow.jobs.job import Job
from airflow.serialization.pydantic.job import JobPydantic
-J =
klexadoc commented on issue #34065:
URL: https://github.com/apache/airflow/issues/34065#issuecomment-1706464703
Confirmed that https://github.com/apache/airflow/issues/33478 is fixed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk merged PR #34094:
URL: https://github.com/apache/airflow/pull/34094
--
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:
potiuk merged PR #33898:
URL: https://github.com/apache/airflow/pull/33898
--
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:
Lee-W opened a new pull request, #34104:
URL: https://github.com/apache/airflow/pull/34104
Update missing content in
https://airflow.apache.org/docs/apache-airflow-providers-microsoft-azure/stable/connections/azure_container_volume.html
---
**^
ephraimbuddy commented on issue #34023:
URL: https://github.com/apache/airflow/issues/34023#issuecomment-1706432286
@hussein-awala thanks for debugging, I was busy with releases the past 2
days, getting to looking at it now
--
This is an automated message from the Apache Git Service.
vidyapanchalZS opened a new issue, #34105:
URL: https://github.com/apache/airflow/issues/34105
### Apache Airflow version
Other Airflow 2 version (please specify below)
### What happened
Hello team,
We are facing one issue on Airflow version 2.2.2 where
josh-fell opened a new pull request, #34106:
URL: https://github.com/apache/airflow/pull/34106
Best practice to is make constructors of operators/sensors as simple as
possible. Currently this sensor directly builds a KubernetesHook object in its
construction. Yes, there are db or external
potiuk closed issue #33178: Flaky `test_xcom_map_error_fails_task` test
URL: https://github.com/apache/airflow/issues/33178
--
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
potiuk commented on PR #33309:
URL: https://github.com/apache/airflow/pull/33309#issuecomment-1706741295
Thanks @uranusjr !
--
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.
potiuk commented on PR #34031:
URL: https://github.com/apache/airflow/pull/34031#issuecomment-1703851208
I think you really can't event have another 5.3.2 release. PyPi will not let
you upload another 5.3.2 even if you delete the previous one
--
This is an automated message from the
Bisk1 opened a new pull request, #34037:
URL: https://github.com/apache/airflow/pull/34037
When unit tests are run on local virtual env on macOS, a lot of them fail
because mocks do not work as expected when they are used in subprocesses. See
Nusnus commented on PR #34031:
URL: https://github.com/apache/airflow/pull/34031#issuecomment-1703860020
@hussein-awala @potiuk
With every failure, we learn new lessons.
As the most recent Owner at Celery, I am still discovering what's under the
hood and I've been
amoghrajesh commented on code in PR #34004:
URL: https://github.com/apache/airflow/pull/34004#discussion_r1313873764
##
dev/breeze/src/airflow_breeze/params/doc_build_params.py:
##
@@ -42,6 +55,9 @@ def args_doc_builder(self) -> list[str]:
hussein-awala commented on code in PR #33841:
URL: https://github.com/apache/airflow/pull/33841#discussion_r1314021479
##
tests/serialization/test_serialized_objects.py:
##
@@ -39,10 +39,9 @@ def test_recursive_serialize_calls_must_forward_kwargs():
class_def = None
pankajkoti commented on issue #34022:
URL: https://github.com/apache/airflow/issues/34022#issuecomment-1704004495
you can keep the constraints as is and just install the newer azure provider
at the end of your image build process
--
This is an automated message from the Apache Git
Taragolis opened a new pull request, #34046:
URL: https://github.com/apache/airflow/pull/34046
I'm not sure what is happen yet but upgrade to latests moto==4.2.2 was break
our tests in main, this change make our tests work either with latest `moto`
and previous versions
NJ-Tokki opened a new issue, #34048:
URL: https://github.com/apache/airflow/issues/34048
### Description
Feature
Kubernetes offers a [list of recommended
labels](https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels)
for grouping resource
Bisk1 opened a new pull request, #34049:
URL: https://github.com/apache/airflow/pull/34049
Follow-up to discussion in
https://github.com/apache/airflow/pull/34027#pullrequestreview-1608444868
Some variables used when invoking calculate_dagrun_date_fields were renamed
to make it
Bisk1 commented on PR #34027:
URL: https://github.com/apache/airflow/pull/34027#issuecomment-1704264908
> Looks good but the renaming should be in a separate PR if it's actually
needed
@ephraimbuddy
Ok I removed it and moved to separate PR
hussein-awala commented on PR #33903:
URL: https://github.com/apache/airflow/pull/33903#issuecomment-1704326323
I understand that `+602 −236 ` is scary for a patch version, but most of the
changed code is documentation and tests, and for the changes in
`trigger_rule_dep`, @dstandish has
ephraimbuddy merged PR #33903:
URL: https://github.com/apache/airflow/pull/33903
--
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:
eumiro opened a new pull request, #34052:
URL: https://github.com/apache/airflow/pull/34052
(no comment)
--
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,
potiuk merged PR #33839:
URL: https://github.com/apache/airflow/pull/33839
--
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:
1 - 100 of 64290 matches
Mail list logo