This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard a36fb6eb82 Create temporary virtualenv as context manager
discard ff0fc7568a Respect docker-tests re
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new ac50669c82 update to latest service bus (#38384)
a
boring-cyborg[bot] commented on PR #38384:
URL: https://github.com/apache/airflow/pull/38384#issuecomment-2016376503
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 contributions.
eladkal merged PR #38384:
URL: https://github.com/apache/airflow/pull/38384
--
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.
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536239715
##
airflow/providers/google/cloud/hooks/automl.py:
##
@@ -1,642 +0,0 @@
-#
Review Comment:
Technically speaking - we didn't actually deprecate this hook, but as G
jalengg commented on code in PR #38398:
URL: https://github.com/apache/airflow/pull/38398#discussion_r1536565254
##
airflow/providers/google/cloud/hooks/gcs.py:
##
@@ -1006,6 +1006,27 @@ def get_md5hash(self, bucket_name: str, object_name:
str) -> str:
self.log.info("T
sudohainguyen commented on issue #37681:
URL: https://github.com/apache/airflow/issues/37681#issuecomment-2016329205
hey can we expect this will be fixed in the next release?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
dirrao commented on code in PR #38398:
URL: https://github.com/apache/airflow/pull/38398#discussion_r1536551567
##
airflow/providers/google/cloud/hooks/gcs.py:
##
@@ -1006,6 +1006,27 @@ def get_md5hash(self, bucket_name: str, object_name:
str) -> str:
self.log.info("Th
dirrao commented on code in PR #38408:
URL: https://github.com/apache/airflow/pull/38408#discussion_r1536548308
##
airflow/providers/google/cloud/operators/bigquery.py:
##
@@ -322,8 +322,26 @@ def execute(self, context: Context):
),
meth
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
from ff0fc7568a Respect docker-tests requirements (use venv)
add a36fb6eb82 Create temporary virtuale
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
from fdbb7e3d37 Use `python-on-whales` in docker tests
add dec65d5400 Use image fixture for determine
lyuyunbo commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2016229746
> ```
> change()
> try:
>do stuff
> finally:
>restore()
> ```
Ha! I misunderstood which try-catch block you were mentioning before.
Updated, this is ind
kaxil commented on issue #34013:
URL: https://github.com/apache/airflow/issues/34013#issuecomment-2016216787
>Can everyone in this thread make an experiment, disable it and report back
here - at least this will give some clue on where the problems might happen.
Yes please, @alex-magno
Lee-W commented on PR #38389:
URL: https://github.com/apache/airflow/pull/38389#issuecomment-2016215062
> related to that #38353 ?
I think it's somewhat related but not 100% sure this is the root cause
--
This is an automated message from the Apache Git Service.
To respond to the me
github-actions[bot] commented on PR #35639:
URL: https://github.com/apache/airflow/pull/35639#issuecomment-2016205670
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for you
github-actions[bot] commented on PR #35966:
URL: https://github.com/apache/airflow/pull/35966#issuecomment-2016205635
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for you
github-actions[bot] commented on PR #36319:
URL: https://github.com/apache/airflow/pull/36319#issuecomment-2016205587
This pull request has been automatically marked as stale because it has not
had recent activity. It will be closed in 5 days if no further activity occurs.
Thank you for you
Taragolis opened a new pull request, #38421:
URL: https://github.com/apache/airflow/pull/38421
This attempt to reduce number of subprocess usage and switch to
[`python-on-whales`](https://github.com/gabrieldemarmiesse/python-on-whales)
where it possible.
The main bene
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit fdbb7e3d37fad146a4dc16f007dde58d0fbeae69
Author: Andrey Anshin
AuthorDate: Sat Mar 23 03:40:48 2024 +04
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard 393af8e7bf Use `python-on-whales` in docker tests
new fdbb7e3d37 Use `python-on-whales` in docke
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit d9549bbadeee2193037e4a2ae88bf255a315bb09
Author: Andrey Anshin
AuthorDate: Sat Mar 23 03:40:48 2024 +04
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
at d9549bbade Use `python-on-whales` in docker tests
This branch includes the following new commits:
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 393af8e7bfcfe72d4500bb25153fe6228bf7d017
Author: Andrey Anshin
AuthorDate: Sat Mar 23 03:40:48 2024 +04
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch modify-docker-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard d9549bbade Use `python-on-whales` in docker tests
new 393af8e7bf Use `python-on-whales` in docke
potiuk commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2016154715
```
change()
try:
do stuff
finally:
restore()
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
kaxil commented on PR #38401:
URL: https://github.com/apache/airflow/pull/38401#issuecomment-2016148389
>If it does, and it results in a slow migration, I really wonder if we
should not do this and instruct users to set up an object storage xcom backend
instead. Thoughts?
The current
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new 3aeee405a2 Updating con
hussein-awala commented on PR #37989:
URL: https://github.com/apache/airflow/pull/37989#issuecomment-2016109464
The PR was broken with around 100 unrelated commits, I just squashed all
related commits into a single commit and rebased `main` as it was impossible to
review the PR. Could you p
lyuyunbo commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2016091345
> Still try/finally not solved
Can you be more specific please @potiuk? Because we indeed added
`_restore_log_formatter` into the except clause, which will revert the changes
if
Satoshi-Sh commented on PR #37638:
URL: https://github.com/apache/airflow/pull/37638#issuecomment-2016090236
I've been working on `tests/api_connexion/test_cors.py`. It would be great
if you could give me some inputs on this.
You can find the details
[here](https://github.com/sudipto
andyguwc commented on PR #38001:
URL: https://github.com/apache/airflow/pull/38001#issuecomment-2016065152
@josh-fell As I think about adding a flag for retry from failure. I can't
wrap my head around the logic for retry in a deferrable operator. Mind share
your thoughts on how this could w
potiuk commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2016055030
Still try/finally not solved
--
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
simond opened a new issue, #38420:
URL: https://github.com/apache/airflow/issues/38420
### Description
When clicking on a task in the grid view, a newly-created chart that shows
the duration of the tasks shows two reference lines. It is not clear what these
reference lines mean and t
potiuk commented on issue #38347:
URL: https://github.com/apache/airflow/issues/38347#issuecomment-2016047870
> What's currently being done for connections and variables to solve that
issue? They both already have declarative equivalents.
No - they don't - same problem. When you allow
potiuk commented on issue #38347:
URL: https://github.com/apache/airflow/issues/38347#issuecomment-2016046849
> How have they failed?
To solve all the edge cases and the fact that users can modify the DB after
it has been created and how to consistently mantain source of the truth.t B
potiuk commented on PR #37937:
URL: https://github.com/apache/airflow/pull/37937#issuecomment-2016043868
@ashb ?
--
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 unsubscr
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from a7d7ef6433 Don't allow defaults other than None in context parameters,
and improve error message (#38015)
add 88e
potiuk merged PR #38419:
URL: https://github.com/apache/airflow/pull/38419
--
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
lyuyunbo commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2015985430
> conflicts to resolve when rebasing.
Resolved.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
potiuk commented on PR #38259:
URL: https://github.com/apache/airflow/pull/38259#issuecomment-2015984516
> what do you mean by breaking IMHO it's just broken, not breaking - because
it's impossible to foresee how one pod executing will impact another one, and
since you have no influence on
potiuk commented on PR #38008:
URL: https://github.com/apache/airflow/pull/38008#issuecomment-2015980402
conflicts to resolve when rebasing.
--
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 sp
boring-cyborg[bot] commented on PR #38015:
URL: https://github.com/apache/airflow/pull/38015#issuecomment-2015977471
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 contributions.
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new a7d7ef6433 Don't allow defaults other than None in
potiuk closed issue #38006: Context variable defaults break tasks where these
parameters don't go at the end of the task fuction definition
URL: https://github.com/apache/airflow/issues/38006
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
potiuk merged PR #38015:
URL: https://github.com/apache/airflow/pull/38015
--
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
This is an automated email from the ASF dual-hosted git repository.
husseinawala pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 30817a5c6d support iam token from metadata, s
hussein-awala merged PR #38411:
URL: https://github.com/apache/airflow/pull/38411
--
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...@ai
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536239715
##
airflow/providers/google/cloud/hooks/automl.py:
##
@@ -1,642 +0,0 @@
-#
Review Comment:
Technically speaking - we didn't actually deprecate this hook, but as G
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536239715
##
airflow/providers/google/cloud/hooks/automl.py:
##
@@ -1,642 +0,0 @@
-#
Review Comment:
Technically speaking - we didn't actually deprecate this hook, but as G
potiuk opened a new pull request, #38419:
URL: https://github.com/apache/airflow/pull/38419
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pul
shahar1 commented on code in PR #38418:
URL: https://github.com/apache/airflow/pull/38418#discussion_r1536239715
##
airflow/providers/google/cloud/hooks/automl.py:
##
@@ -1,642 +0,0 @@
-#
Review Comment:
Technically speaking - we didn't actually deprecate this hook, but as G
potiuk merged PR #38415:
URL: https://github.com/apache/airflow/pull/38415
--
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
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new fd5fe8d2c6 Revert ObjectStorage config variables na
shahar1 opened a new pull request, #38418:
URL: https://github.com/apache/airflow/pull/38418
Starting March 31, 2024, GCP will sunset the old AutoML API, thus making the
deprecated operators and hook effectively unusable. This PR removes all related
logic (operators, hook an
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 9b2d8e0fcd Upgrade to latest chart dependencies detected by canary run
(#38416)
add bcd7d35ff8 Update Airflow dec
potiuk merged PR #38406:
URL: https://github.com/apache/airflow/pull/38406
--
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
shahar1 opened a new pull request, #38417:
URL: https://github.com/apache/airflow/pull/38417
closes: #37400
The following line wrongly parsed the `parent_model` param in all Vertex AI
AutoML "Create" operators:
```python
self.parent_model = self.parent_model.rpar
wircho commented on issue #21900:
URL: https://github.com/apache/airflow/issues/21900#issuecomment-2015828679
@yeachan153 Did you ever solve this problem? We would love to be able to
keep pods running during environment restarts, and it looks like your idea
might work.
--
This is an auto
potiuk commented on PR #38281:
URL: https://github.com/apache/airflow/pull/38281#issuecomment-2015813508
Probably yes. How to perform that?
> Just the usual "Refactor -> Extract" - both are in common.sql provider, so
adding `common.sql.util` package with `provider_compatibility.py` a
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 9b2d8e0fcd Upgrade to latest chart dependencies det
potiuk merged PR #38416:
URL: https://github.com/apache/airflow/pull/38416
--
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
potiuk commented on PR #38401:
URL: https://github.com/apache/airflow/pull/38401#issuecomment-2015810027
> > I think, if that's the case then we will likely have to at least warn
the users in significant note that it might happen and that they should likely
run db clean on xcom table for so
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 0d11f3ca2f Add experimental warning in AWS auth
Taragolis merged PR #38414:
URL: https://github.com/apache/airflow/pull/38414
--
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...@airflo
frodo2000 commented on PR #38281:
URL: https://github.com/apache/airflow/pull/38281#issuecomment-2015774537
> How about extracting the common code for version check? Sounds reasonable
to do it now?
Probably yes. How to perform that?
--
This is an automated message from the Apache G
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 36704fb9a7 Fix release docs for the helm chart (#38
potiuk merged PR #38413:
URL: https://github.com/apache/airflow/pull/38413
--
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
potiuk opened a new pull request, #38416:
URL: https://github.com/apache/airflow/pull/38416
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pul
pankajastro commented on PR #38142:
URL: https://github.com/apache/airflow/pull/38142#issuecomment-2015737146
@bolkedebruin create a PR https://github.com/apache/airflow/pull/38415 PTAL
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
pankajastro opened a new pull request, #38415:
URL: https://github.com/apache/airflow/pull/38415
Address https://github.com/apache/airflow/pull/38142#issuecomment-2007166415
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](h
potiuk commented on PR #37638:
URL: https://github.com/apache/airflow/pull/37638#issuecomment-2015695912
> ### Does not raise Valueerror
When I run it in my unit test, I got Internal Server error - that's why it
does not return ValueError, because it converts ValueError into 500 inter
ephraimbuddy commented on PR #38094:
URL: https://github.com/apache/airflow/pull/38094#issuecomment-2015691919
> > Nice. The prefix is cool.
> > There is one worry with that U just wanted to mention it.
> > I understand that we are masking secreates before serialization ? Just
wanted t
ephraimbuddy commented on PR #38094:
URL: https://github.com/apache/airflow/pull/38094#issuecomment-2015670623
> Nice. The prefix is cool.
>
> There is one worry with that U just wanted to mention it.
>
> I understand that we are masking secreates before serialization ? Just
wa
ketozhang commented on issue #38347:
URL: https://github.com/apache/airflow/issues/38347#issuecomment-2015651861
Thanks and yep, it's a classic problem most devops folks have to deal with.
It's heavily dependent if you want to lock down configs post-deploy and if DB
should be overwritten du
potiuk commented on issue #34013:
URL: https://github.com/apache/airflow/issues/34013#issuecomment-2015639112
No. I do not see anyone working on it nor providing a solution. But the log
above suggests that this is caused by so called "mini scheduler" which is
enabled by this configuration
potiuk merged PR #38412:
URL: https://github.com/apache/airflow/pull/38412
--
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
potiuk merged PR #38362:
URL: https://github.com/apache/airflow/pull/38362
--
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
sudiptob2 commented on PR #37638:
URL: https://github.com/apache/airflow/pull/37638#issuecomment-2015632855
> Since is it API experimental (which is deprecated), I would not spend too
much time on it. Maybe change the expected exception? Change it to expect
`ValueError` from `connexion` lib
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 6a225ccb2a to avoid type's mismatch durind concaten
boring-cyborg[bot] commented on PR #38412:
URL: https://github.com/apache/airflow/pull/38412#issuecomment-2015621212
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 contributions.
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 5344881715 Make postgresql default engine args comp
potiuk closed issue #38064: Airflow postgres executemany_mode': 'values' error
URL: https://github.com/apache/airflow/issues/38064
--
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 code in PR #38413:
URL: https://github.com/apache/airflow/pull/38413#discussion_r1535975928
##
dev/chart/build_changelog_annotations.py:
##
@@ -98,7 +98,7 @@ def print_entry(section: str, description: str, pr_number:
int | None):
in_first_releas
enchant3dmango commented on code in PR #38413:
URL: https://github.com/apache/airflow/pull/38413#discussion_r1535967153
##
dev/chart/build_changelog_annotations.py:
##
@@ -98,7 +98,7 @@ def print_entry(section: str, description: str, pr_number:
int | None):
in_firs
brokenjacobs commented on issue #38382:
URL: https://github.com/apache/airflow/issues/38382#issuecomment-2015577073
LGTM!, keda bypass of pgbouncer is working as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 947c48b2fb DockerOperator: use DOCKER_HOST as default for docker_url
(#38387)
add 40559a315e Fix typo in verifica
potiuk merged PR #38410:
URL: https://github.com/apache/airflow/pull/38410
--
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
potiuk commented on issue #38347:
URL: https://github.com/apache/airflow/issues/38347#issuecomment-2015572014
Sure. If you have a good idea how to do it in a robust way - proposing a
solution, especially one that will not overwrite existing DB and will allow to
both declaratively upload rol
vincbeck commented on PR #37638:
URL: https://github.com/apache/airflow/pull/37638#issuecomment-2015557108
> ### Does not raise Valueerror
> This test seems not to raise `ValueError` anymore after migrating to
connexion v3. See **screenshot 1**. But a ValueError is indeed raised from deep
brokenjacobs commented on issue #38382:
URL: https://github.com/apache/airflow/issues/38382#issuecomment-2015552750
quay.io outage causing issues with the statsd container:
![image](https://github.com/apache/airflow/assets/3159463/562b91ea-2813-435e-80fb-396f489c00f2)
```
Warning
vincbeck opened a new pull request, #38414:
URL: https://github.com/apache/airflow/pull/38414
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#p
ephraimbuddy commented on PR #36462:
URL: https://github.com/apache/airflow/pull/36462#issuecomment-2015546025
Can you add this test:
```
def
test_mapped_tasks_in_mapped_task_group_waits_for_upstreams_to_complete(dag_maker,
session):
"""Test that one failed trigger rule works w
jedcunningham opened a new pull request, #38413:
URL: https://github.com/apache/airflow/pull/38413
(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 unsubscri
potiuk commented on PR #38281:
URL: https://github.com/apache/airflow/pull/38281#issuecomment-2015543690
How about extracting the common code for version check? Sounds reasonable to
do it now?
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
igeni opened a new pull request, #38412:
URL: https://github.com/apache/airflow/pull/38412
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull
boring-cyborg[bot] commented on PR #38412:
URL: https://github.com/apache/airflow/pull/38412#issuecomment-2015543056
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 Contributors'
potiuk commented on code in PR #38281:
URL: https://github.com/apache/airflow/pull/38281#discussion_r1535923249
##
airflow/providers/common/sql/transfers/sql.py:
##
@@ -0,0 +1,248 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license
ephraimbuddy commented on code in PR #37143:
URL: https://github.com/apache/airflow/pull/37143#discussion_r1535920258
##
airflow/sentry/__init__.py:
##
@@ -0,0 +1,30 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. S
uzhastik opened a new pull request, #38411:
URL: https://github.com/apache/airflow/pull/38411
Add ability to use IAM token from metadata service in YQ operator
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.
jedcunningham commented on code in PR #38094:
URL: https://github.com/apache/airflow/pull/38094#discussion_r1535918174
##
airflow/serialization/helpers.py:
##
@@ -38,6 +39,13 @@ def is_jsonable(x):
else:
return True
+max_length = conf.getint("core", "
1 - 100 of 266 matches
Mail list logo