uranusjr commented on pull request #20664:
URL: https://github.com/apache/airflow/pull/20664#issuecomment-1005457445
I’m not too familiar with buildx in the first palce and will dismiss my
request for review.
--
This is an automated message from the Apache Git Service.
To respond to the
github-actions[bot] commented on pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#issuecomment-1005457194
The PR is likely ready to be merged. No tests are needed as no important
environment files, nor python files were modified by it. However, committers
might decide
potiuk commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778608790
##
File path: dev/breeze/src/airflow_breeze/breeze.py
##
@@ -95,12 +93,103 @@ def shell(verbose: bool):
@option_verbose
-@main.command()
-def build_
uranusjr opened a new pull request #20669:
URL: https://github.com/apache/airflow/pull/20669
Tests are failing:
```
TypeError: Protocols can only inherit from other protocols, got
```
This is due to the compatibility shim for `ECSProtocol` is incorrectly
double-inheriti
This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from b83084b Add filter by state in DagRun REST API (List Dag Runs)
(#20485)
add 64c0bd5 bugfix: deferred
ephraimbuddy closed issue #20580:
URL: https://github.com/apache/airflow/issues/20580
--
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-unsu
ephraimbuddy merged pull request #20649:
URL: https://github.com/apache/airflow/pull/20649
--
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
eladkal commented on a change in pull request #20360:
URL: https://github.com/apache/airflow/pull/20360#discussion_r778603523
##
File path: airflow/providers/amazon/aws/hooks/dynamodb.py
##
@@ -18,13 +18,14 @@
"""This module contains the AWS DynamoDB hook"""
+import warnin
eladkal commented on a change in pull request #20369:
URL: https://github.com/apache/airflow/pull/20369#discussion_r778602745
##
File path: airflow/providers/amazon/aws/operators/batch.py
##
@@ -26,17 +26,18 @@
- http://boto3.readthedocs.io/en/latest/reference/services/bat
pingzh commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778598911
##
File path: tests/utils/test_operator_helpers.py
##
@@ -71,6 +71,29 @@ def test_context_to_airflow_vars_all_context(self):
'AIRFLOW_CTX_DA
Bowrna commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778596878
##
File path: dev/breeze/src/airflow_breeze/breeze.py
##
@@ -95,12 +93,103 @@ def shell(verbose: bool):
@option_verbose
-@main.command()
-def build_
uranusjr commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778592427
##
File path: tests/utils/test_operator_helpers.py
##
@@ -71,6 +71,29 @@ def test_context_to_airflow_vars_all_context(self):
'AIRFLOW_CTX_
potiuk commented on pull request #20664:
URL: https://github.com/apache/airflow/pull/20664#issuecomment-1005438706
cc: @Bowrna @edithturn - > that will make your job much easier, it
simplifies a lot of the "caching" complexity - that's why I did not want you to
start looking at it before a
uranusjr commented on pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#issuecomment-1005438826
Oh, the last one is from your change though; you’re using `with
pytest.raises(TypeError) as error` incorrectly.
--
This is an automated message from the Apache Git Service
uranusjr commented on pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#issuecomment-1005437824
Looks unrelated, probably someone merged some bad code to main. I’ll
investigate separately.
--
This is an automated message from the Apache Git Service.
To respond to the
potiuk closed pull request #20238:
URL: https://github.com/apache/airflow/pull/20238
--
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-unsub
potiuk commented on pull request #20664:
URL: https://github.com/apache/airflow/pull/20664#issuecomment-1005437311
This is a VAST simplification of our docker build caching (using modern
BuildKit plugin). I have to do some more testing with it, but on top of
removing several hundreds of li
uranusjr commented on a change in pull request #20651:
URL: https://github.com/apache/airflow/pull/20651#discussion_r778589556
##
File path: airflow/providers/imap/hooks/imap.py
##
@@ -71,7 +71,7 @@ def get_conn(self) -> 'ImapHook':
"""
if not self.mail_client
Greetlist commented on issue #20308:
URL: https://github.com/apache/airflow/issues/20308#issuecomment-1005435631
At this moment, I solve this problem by this:
![image](https://user-images.githubusercontent.com/18439743/148174419-5f27d495-ef6c-4a08-b262-c41ada4725ad.png)
Pass *
This is an automated email from the ASF dual-hosted git repository.
uranusjr pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 9c0ba1b Standardize AWS ECS naming (#20332)
add b83084b Add filter by state in DagRun REST API (List Dag Runs
potiuk opened a new pull request #20664:
URL: https://github.com/apache/airflow/pull/20664
The "buildkit" is much more modern docker build mechanism and supports
multiarchitecture builds which makes it suitable for our future ARM
support, it also has nicer UI and much more sophisticate
pingzh edited a comment on pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#issuecomment-1005433855
@potiuk @uranusjr any ideas about this error?
https://github.com/apache/airflow/runs/4711391810?check_suite_focus=true#step:8:8255
![image](https://user-images
pingzh edited a comment on pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#issuecomment-1005433855
@potiuk @uranusjr any ideas about this error?
https://github.com/apache/airflow/runs/4711391810?check_suite_focus=true#step:8:8255
![image](https://user-images.g
pingzh commented on pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#issuecomment-1005433855
@potiuk @uranusjr any ideas about this error?
![image](https://user-images.githubusercontent.com/8662365/148174829-d9be104f-773e-44df-8243-3a24f36f72a3.png)
I hav
uranusjr merged pull request #20485:
URL: https://github.com/apache/airflow/pull/20485
--
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-uns
boring-cyborg[bot] commented on pull request #20485:
URL: https://github.com/apache/airflow/pull/20485#issuecomment-1005429527
Awesome work, congrats on your first merged pull request!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
pingzh commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778558416
##
File path: airflow/utils/operator_helpers.py
##
@@ -77,6 +87,21 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
github-actions[bot] commented on pull request #20656:
URL: https://github.com/apache/airflow/pull/20656#issuecomment-1005398080
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and ta
uranusjr commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778554258
##
File path: airflow/utils/operator_helpers.py
##
@@ -77,6 +87,21 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
github-actions[bot] commented on pull request #20657:
URL: https://github.com/apache/airflow/pull/20657#issuecomment-1005392592
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and ta
jedcunningham closed pull request #20661:
URL: https://github.com/apache/airflow/pull/20661
--
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: commit
uranusjr commented on issue #20320:
URL: https://github.com/apache/airflow/issues/20320#issuecomment-1005381988
Go ahead.
--
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.
T
github-actions[bot] commented on pull request #20663:
URL: https://github.com/apache/airflow/pull/20663#issuecomment-1005381486
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
arsalanfardi commented on issue #20320:
URL: https://github.com/apache/airflow/issues/20320#issuecomment-1005381384
Can I take this up as my first 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
uranusjr commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778544900
##
File path: airflow/decorators/sensor.py
##
@@ -0,0 +1,89 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor lic
kaxil commented on pull request #20663:
URL: https://github.com/apache/airflow/pull/20663#issuecomment-1005377013
@Jorricks Can you take a look at it please? This PR still respects the table
you had in the PR description for https://github.com/apache/airflow/pull/16634
but allow viewing re
kaxil opened a new pull request #20663:
URL: https://github.com/apache/airflow/pull/20663
This was updated in Airflow 2.2.0 via
https://github.com/apache/airflow/pull/16634 which restricts a user to even
views the DagRuns and TI records if they don't have "edit" permissions on DAG
even th
pingzh commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778523450
##
File path: airflow/utils/operator_helpers.py
##
@@ -75,6 +86,19 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
uranusjr commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778521753
##
File path: airflow/utils/operator_helpers.py
##
@@ -75,6 +86,16 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
uranusjr commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778521582
##
File path: airflow/utils/operator_helpers.py
##
@@ -75,6 +86,19 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
github-actions[bot] commented on pull request #20659:
URL: https://github.com/apache/airflow/pull/20659#issuecomment-1005347651
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
github-actions[bot] commented on pull request #19965:
URL: https://github.com/apache/airflow/pull/19965#issuecomment-1005326870
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and ta
pingzh commented on a change in pull request #20361:
URL: https://github.com/apache/airflow/pull/20361#discussion_r778486378
##
File path: airflow/utils/operator_helpers.py
##
@@ -75,6 +86,16 @@ def context_to_airflow_vars(context: Mapping[str, Any],
in_env_var_format: bool
Bowrna commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778483376
##
File path: dev/breeze/src/airflow_breeze/cache.py
##
@@ -0,0 +1,67 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contrib
jedcunningham opened a new pull request #20661:
URL: https://github.com/apache/airflow/pull/20661
Add changelog for chart version 1.4.0.
--
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 speci
kazanzhy commented on a change in pull request #20642:
URL: https://github.com/apache/airflow/pull/20642#discussion_r778472926
##
File path: airflow/providers/amazon/aws/hooks/rds.py
##
@@ -0,0 +1,165 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or mor
kazanzhy removed a comment on pull request #20642:
URL: https://github.com/apache/airflow/pull/20642#issuecomment-1005276831
A lot of thanks that clarifying the way to implementation @dstandish
The solutions below don't work and it's expected
```
from typing import Type
kazanzhy commented on a change in pull request #20642:
URL: https://github.com/apache/airflow/pull/20642#discussion_r778472926
##
File path: airflow/providers/amazon/aws/hooks/rds.py
##
@@ -0,0 +1,165 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or mor
kazanzhy commented on pull request #20642:
URL: https://github.com/apache/airflow/pull/20642#issuecomment-1005276831
A lot of thanks that clarifying the way to implementation @dstandish
The solutions below don't work and it's expected
```
from typing import Type
from bo
ferruzzi closed pull request #20369:
URL: https://github.com/apache/airflow/pull/20369
--
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-uns
ferruzzi commented on pull request #20374:
URL: https://github.com/apache/airflow/pull/20374#issuecomment-1005268141
This one is throwing the Protocol inheritance error that I mentioned in the
other PR:
https://github.com/apache/airflow/runs/4708697673?check_suite_focus=true#step:8:8489
ferruzzi edited a comment on pull request #20374:
URL: https://github.com/apache/airflow/pull/20374#issuecomment-1005258276
rebased and I think I covered
https://github.com/apache/airflow/pull/20374#discussion_r771918489
--
This is an automated message from the Apache Git Service.
To res
ferruzzi commented on pull request #20374:
URL: https://github.com/apache/airflow/pull/20374#issuecomment-1005258276
rebased and I think I covered
https://github.com/apache/airflow/pull/20374#discussion_r771535786
--
This is an automated message from the Apache Git Service.
To respond to
ferruzzi commented on a change in pull request #20360:
URL: https://github.com/apache/airflow/pull/20360#discussion_r778457959
##
File path: airflow/providers/amazon/aws/hooks/dynamodb.py
##
@@ -18,13 +18,14 @@
"""This module contains the AWS DynamoDB hook"""
+import warni
williamfmr commented on issue #20320:
URL: https://github.com/apache/airflow/issues/20320#issuecomment-1005250016
@potiuk, unfortunately my employer does not allow for contributing code to
external projects. I have removed myself from this ticket. Very sorry :(
--
This is an automated me
yehoshuadimarsky opened a new issue #20660:
URL: https://github.com/apache/airflow/issues/20660
### Apache Airflow Provider(s)
google
### Versions of Apache Airflow Providers
```
apache-airflow-providers-amazon==2.3.0
apache-airflow-providers-celery==2.1.0
apach
dimon222 commented on issue #20615:
URL: https://github.com/apache/airflow/issues/20615#issuecomment-1005238859
#19726 briefly tested, seems to work as intended and not raising any
problems.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
tikikun commented on issue #15635:
URL: https://github.com/apache/airflow/issues/15635#issuecomment-1005236541
Has anyone done 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 specific
subkanthi opened a new pull request #20659:
URL: https://github.com/apache/airflow/pull/20659
closes: #20655
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidel
subkanthi commented on issue #20655:
URL: https://github.com/apache/airflow/issues/20655#issuecomment-1005214394
I can take a look at this to disable the edit option.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
ferruzzi opened a new pull request #20369:
URL: https://github.com/apache/airflow/pull/20369
Part of https://github.com/apache/airflow/issues/20296
In addition:
- Fixed capitalization in some comments from `batch` to `Batch` where
applicable.
- Changed `execute(None)` and `
ferruzzi closed pull request #20369:
URL: https://github.com/apache/airflow/pull/20369
--
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-uns
turbaszek commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778391201
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
raise
ferruzzi commented on a change in pull request #20369:
URL: https://github.com/apache/airflow/pull/20369#discussion_r778387027
##
File path: airflow/providers/amazon/aws/hooks/batch_client.py
##
@@ -538,3 +538,35 @@ def exp(tries):
delay = 1 + pow(tries * 0.6, 2)
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from f864d14 Fix precedence of affinity, nodeSelector, and tolerations
(#20641)
add 9c0ba1b Standardize AWS ECS na
eladkal merged pull request #20332:
URL: https://github.com/apache/airflow/pull/20332
--
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-unsu
mingshi-wang commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778380857
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
rai
eladkal commented on a change in pull request #20369:
URL: https://github.com/apache/airflow/pull/20369#discussion_r778378157
##
File path: airflow/providers/amazon/aws/hooks/batch_client.py
##
@@ -538,3 +538,35 @@ def exp(tries):
delay = 1 + pow(tries * 0.6, 2)
eladkal closed issue #17010:
URL: https://github.com/apache/airflow/issues/17010
--
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.
eladkal commented on issue #17010:
URL: https://github.com/apache/airflow/issues/17010#issuecomment-1005155935
So I'm closing this issue for now.
If needed we can revisit in the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
github-actions[bot] commented on pull request #20332:
URL: https://github.com/apache/airflow/pull/20332#issuecomment-1005153877
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
turbaszek commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778373053
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
raise
turbaszek commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778373053
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
raise
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 298435a Add a button to set all tasks to skipped (#20455)
add f864d14 Fix precedence of affinity, nodeSe
jedcunningham merged pull request #20641:
URL: https://github.com/apache/airflow/pull/20641
--
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: commit
jedcunningham closed issue #20538:
URL: https://github.com/apache/airflow/issues/20538
--
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-uns
malthe commented on issue #17010:
URL: https://github.com/apache/airflow/issues/17010#issuecomment-1005149267
I think the boolean combination idea is a good one – and probably @potiuk is
right that throwing time in the mix is a bad one.
That said, if it ain't broke don't fix it and w
mingshi-wang commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778344613
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
rai
turbaszek commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778334782
##
File path: docs/apache-airflow/tutorial_taskflow_api.rst
##
@@ -208,6 +208,51 @@ Python version to run your function.
These two options should all
ferruzzi commented on pull request #20332:
URL: https://github.com/apache/airflow/pull/20332#issuecomment-1005107559
Huh. Neither can I now.I've rebased and removed the `Protocol`
inheritance on L546.
--
This is an automated message from the Apache Git Service.
To respond to the me
turbaszek commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778333159
##
File path: airflow/decorators/__init__.py
##
@@ -29,9 +30,11 @@ def __getattr__(self, name):
if name.startswith("__"):
raise
dstandish opened a new pull request #20658:
URL: https://github.com/apache/airflow/pull/20658
The queue's purpose is to track triggers that need to be canceled. I found
the language `to_delete` a bit confusing because for one it does not actually
delete them but cancel them. The deletion
rdeteix commented on issue #16010:
URL: https://github.com/apache/airflow/issues/16010#issuecomment-1005086285
Thx for the response.
I've already tried it.
Which k8s version are you working with ?
I also tried it without the reverse proxy :
```
ingress:
enable
dstandish opened a new pull request #20657:
URL: https://github.com/apache/airflow/pull/20657
The word 'old' could suggest 'triggers we already have' i.e. 'current
triggers' as compared with the 'requested' triggers, or with 'triggers we need
to create' a.k.a. new triggers. However, here
potiuk commented on issue #17010:
URL: https://github.com/apache/airflow/issues/17010#issuecomment-1005074924
Agree with @eladkal
--
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 co
mingshi-wang commented on a change in pull request #20530:
URL: https://github.com/apache/airflow/pull/20530#discussion_r778301416
##
File path: airflow/decorators/sensor.py
##
@@ -0,0 +1,93 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
potiuk commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778300726
##
File path: dev/breeze/src/airflow_breeze/breeze.py
##
@@ -95,12 +93,103 @@ def shell(verbose: bool):
@option_verbose
-@main.command()
-def build_
potiuk commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778300726
##
File path: dev/breeze/src/airflow_breeze/breeze.py
##
@@ -95,12 +93,103 @@ def shell(verbose: bool):
@option_verbose
-@main.command()
-def build_
mingshi-wang opened a new pull request #20656:
URL: https://github.com/apache/airflow/pull/20656
Updated the base sensor operator such that an optional XCom value can be
returned and pushed when the operator is executed. Users only need to override
the poke() method to return a PokeRet
jedcunningham commented on issue #19038:
URL: https://github.com/apache/airflow/issues/19038#issuecomment-1005065182
This should be fixed on 2.2.3 via #19904.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
jedcunningham closed issue #19038:
URL: https://github.com/apache/airflow/issues/19038
--
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-uns
potiuk commented on a change in pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#discussion_r778296569
##
File path: dev/breeze/src/airflow_breeze/cache.py
##
@@ -0,0 +1,67 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contrib
pierrejeambrun commented on a change in pull request #20386:
URL: https://github.com/apache/airflow/pull/20386#discussion_r778291137
##
File path: airflow/providers/apache/beam/operators/beam.py
##
@@ -470,3 +529,144 @@ def on_kill(self) -> None:
job_id=self.da
mobuchowski commented on a change in pull request #20443:
URL: https://github.com/apache/airflow/pull/20443#discussion_r778287426
##
File path: airflow/listeners/spec.py
##
@@ -0,0 +1,49 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor
mobuchowski commented on a change in pull request #20443:
URL: https://github.com/apache/airflow/pull/20443#discussion_r778286692
##
File path: airflow/jobs/local_task_job.py
##
@@ -71,6 +74,8 @@ def __init__(
# terminate multiple times
self.terminating = Fals
Bowrna commented on pull request #20645:
URL: https://github.com/apache/airflow/pull/20645#issuecomment-1005051028
@potiuk Could you also share your feedback on the files modified? This PR is
still in draft state, but your feedback at this point would be useful
--
This is an automated me
Bowrna commented on pull request #20338:
URL: https://github.com/apache/airflow/pull/20338#issuecomment-1005049302
> Loooks very, very close @Bowrna - just two comments left, I can merge and
then I can add test case example
@potiuk Could you share your reviews on the last two comment
This is an automated email from the ASF dual-hosted git repository.
bbovenzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 27fcd7e [New] - Ci free space python (#20200)
add 298435a Add a button to set all tasks to skipped (#20455)
boring-cyborg[bot] commented on pull request #20455:
URL: https://github.com/apache/airflow/pull/20455#issuecomment-1005045964
Awesome work, congrats on your first merged pull request!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
bbovenzi merged pull request #20455:
URL: https://github.com/apache/airflow/pull/20455
--
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-uns
1 - 100 of 233 matches
Mail list logo