eladkal commented on code in PR #28631:
URL: https://github.com/apache/airflow/pull/28631#discussion_r1058527580
##
tests/providers/conftest.py:
##
@@ -0,0 +1,75 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the
jedcunningham opened a new pull request, #28632:
URL: https://github.com/apache/airflow/pull/28632
This solves a number of issues with our existing (new) audit logging on
things like variables and connections.
Still a WIP, need to verify and add tests for other ViewModels inheriting
blag commented on issue #28280:
URL: https://github.com/apache/airflow/issues/28280#issuecomment-1366839468
TIL. What an odd choice.
While there are many tools that will happily deal with all of the different
forms of newlines, sadly some tools that remain in popular usage still do no
blag closed issue #28280: Inconsistent handling of newlines in tasks
URL: https://github.com/apache/airflow/issues/28280
--
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 unsu
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 761aa59f92 Fix Incorrect Example (#28609)
add 8a23bf47a4 Add note for pymssql requirement (#28625)
No new revi
kaxil merged PR #28625:
URL: https://github.com/apache/airflow/pull/28625
--
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.ap
Taragolis opened a new pull request, #28631:
URL: https://github.com/apache/airflow/pull/28631
Follow-up:
https://github.com/apache/airflow/pull/28337#issuecomment-1348771336
Some hacks which uses in PR
1. `module` scope with parsing "provider name" instead of `package` scope.
pyt
potiuk commented on PR #28630:
URL: https://github.com/apache/airflow/pull/28630#issuecomment-1366832706
The list of "other" folders now is far shorter:
![image](https://user-images.githubusercontent.com/595491/209854635-3efcf2f6-2bee-4028-82b1-8ecfa0256015.png)
Comparing t
potiuk opened a new pull request, #28630:
URL: https://github.com/apache/airflow/pull/28630
The "Other" test category automatically selects all tests that are not
included in any of the regular categories. That is to make sure that we do not
forget to add any directory that has been added.
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 ae05c23304 Updating con
ephraimbuddy commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058500304
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
kaxil commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058492281
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
potiuk commented on PR #28627:
URL: https://github.com/apache/airflow/pull/28627#issuecomment-1366802671
Does not seem to have any bad side effect (let's see what tests say too)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
potiuk opened a new pull request, #28627:
URL: https://github.com/apache/airflow/pull/28627
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
for more info
Taragolis commented on PR #28337:
URL: https://github.com/apache/airflow/pull/28337#issuecomment-1366787812
I tried to do it locally. It required some additional logic in fixture
because for some unknown reason it not working well on "package" scope.
Need to find this branch, complete
potiuk commented on PR #28020:
URL: https://github.com/apache/airflow/pull/28020#issuecomment-1366782782
Could you please update the description and title of the PR (and commit) if
you want to get it reviewed? As it is now, I had no idea what should be
reviewed because neither title nor des
potiuk commented on PR #28337:
URL: https://github.com/apache/airflow/pull/28337#issuecomment-1366781416
> I thought this is possible to achieve by autouse fixture in providers
conftests.py and module/package scope level
Yes. That's exactly what I thought. Would you like to give it a
potiuk commented on PR #28097:
URL: https://github.com/apache/airflow/pull/28097#issuecomment-1366779235
Tests are failing.
--
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.
rkarish commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058460835
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
Taragolis commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058459895
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
potiuk commented on PR #28541:
URL: https://github.com/apache/airflow/pull/28541#issuecomment-1366770329
Anyone needs some speed-ups :) ?
--
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 spec
potiuk commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058456144
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
potiuk commented on code in PR #28619:
URL: https://github.com/apache/airflow/pull/28619#discussion_r1058455261
##
airflow/api/client/json_client.py:
##
@@ -24,9 +24,25 @@
class Client(api_client.Client):
-"""Json API client implementation."""
+"""Json API client im
boring-cyborg[bot] commented on PR #28626:
URL: https://github.com/apache/airflow/pull/28626#issuecomment-1366767160
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
MaicoTimmerman opened a new pull request, #28626:
URL: https://github.com/apache/airflow/pull/28626
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
for m
rkarish commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058450927
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
potiuk commented on code in PR #28619:
URL: https://github.com/apache/airflow/pull/28619#discussion_r1058449396
##
airflow/api/client/json_client.py:
##
@@ -24,9 +24,25 @@
class Client(api_client.Client):
-"""Json API client implementation."""
+"""Json API client im
potiuk commented on issue #22132:
URL: https://github.com/apache/airflow/issues/22132#issuecomment-1366758456
> Is anyone aware that it's being developed? or is likely to be picked up
for development?
Marked it as good first issue. That's what I can do as maintainer.
Just to
Taragolis commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058446218
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
vchiapaikeo commented on PR #28624:
URL: https://github.com/apache/airflow/pull/28624#issuecomment-1366756074
Yup - this latest commit should hopefully fix. Changed the ids as well to
better reflect the action that is taking place.
--
This is an automated message from the Apache Git Servi
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from d4dbb0077a Guard not-yet-expanded ti in trigger rule dep (#28592)
add 761aa59f92 Fix Incorrect Example (#28609)
boring-cyborg[bot] commented on PR #28609:
URL: https://github.com/apache/airflow/pull/28609#issuecomment-1366755472
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 on to GitHub
kaxil merged PR #28609:
URL: https://github.com/apache/airflow/pull/28609
--
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.ap
potiuk commented on PR #28624:
URL: https://github.com/apache/airflow/pull/28624#issuecomment-1366754570
Failing tests :(
--
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 code in PR #28569:
URL: https://github.com/apache/airflow/pull/28569#discussion_r1058441026
##
airflow/providers/slack/notifications/slack_notifier.py:
##
@@ -0,0 +1,87 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor li
stamixthereal commented on code in PR #28619:
URL: https://github.com/apache/airflow/pull/28619#discussion_r1058438631
##
airflow/api/client/json_client.py:
##
@@ -24,9 +24,25 @@
class Client(api_client.Client):
-"""Json API client implementation."""
+"""Json API cl
potiuk commented on issue #25032:
URL: https://github.com/apache/airflow/issues/25032#issuecomment-1366749247
@wangyaopw As @uranusjr mentioned - it's better not to attempt it yet. There
are bugs like just fixed https://github.com/apache/airflow/pull/28592 (which is
likely the fix to your p
potiuk commented on PR #28592:
URL: https://github.com/apache/airflow/pull/28592#issuecomment-1366746841
Nice catch!
--
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 unsu
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 48a051acc4 Minor update to Scheduler documentation (#28620)
add d4dbb0077a Guard not-yet-expanded ti in trigger ru
potiuk merged PR #28592:
URL: https://github.com/apache/airflow/pull/28592
--
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
kaxil commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058434982
##
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##
@@ -91,6 +91,8 @@ PIP package Version required
``pymssql``
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 4f7ac623c8 Fix typo (#28623)
add 48a051acc4 Minor update to Scheduler documentation (#28620)
No new revisions
boring-cyborg[bot] commented on PR #28620:
URL: https://github.com/apache/airflow/pull/28620#issuecomment-1366745647
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 on to GitHub
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 5dcbbd6f17 Fix ``Connection.get_extra`` type (#28594)
add 4f7ac623c8 Fix typo (#28623)
No new revisions were adde
kaxil merged PR #28620:
URL: https://github.com/apache/airflow/pull/28620
--
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.ap
potiuk merged PR #28623:
URL: https://github.com/apache/airflow/pull/28623
--
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.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 4f006e8c71 Add .bash and other extensions to SSHOperator template_ext
(#28617)
add 5dcbbd6f17 Fix ``Connection
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 76186bb588 Fix code docstrings (#28622)
add 4f006e8c71 Add .bash and other extensions to SSHOperator template_ext
kaxil merged PR #28594:
URL: https://github.com/apache/airflow/pull/28594
--
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.ap
boring-cyborg[bot] commented on PR #28594:
URL: https://github.com/apache/airflow/pull/28594#issuecomment-1366743808
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 on to GitHub
potiuk merged PR #28617:
URL: https://github.com/apache/airflow/pull/28617
--
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 #24669:
URL: https://github.com/apache/airflow/issues/24669#issuecomment-1366743181
I think you should then open a new discussion and explain what you tried and
show how it did not work. Commenting on closed issue especially one that
relates to different problem -
potiuk commented on issue #28603:
URL: https://github.com/apache/airflow/issues/28603#issuecomment-1366737663
Of course, we can re-open it if you test both keepalive and migration to
newer version and the issue is still there (though I think some more debugging
info would be needed for that
potiuk closed issue #28603: SFTPOperator does not finish downloading
URL: https://github.com/apache/airflow/issues/28603
--
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 unsu
potiuk commented on issue #28603:
URL: https://github.com/apache/airflow/issues/28603#issuecomment-1366737096
I am closing it provisionally - because I believe keepalive is the solution.
I believe you might also not need to upgrade the provider - because keepalive
seems to be also available
raj-wadhwa commented on issue #22132:
URL: https://github.com/apache/airflow/issues/22132#issuecomment-1366736464
Is there any progress in this area? I was looking for this info to be
returned in the API for DAG Run info:
`ETA` or `Estimated_Time_To_Completion`
Something like this
potiuk commented on issue #28603:
URL: https://github.com/apache/airflow/issues/28603#issuecomment-1366733676
One of the reasons for paramico change was that pysftp did not allow for
the kind of configuration that direct paramiko would. You should likely look at
`keepalive_interval` and po
rkarish opened a new pull request, #28625:
URL: https://github.com/apache/airflow/pull/28625
A user reported an issue on Slack with `MsSqlHook` resulting in a
segmentation fault. This issue was resolved by following the instructions on
the `pymssql` PyPI. I added this note to the documentat
kaxil commented on code in PR #28439:
URL: https://github.com/apache/airflow/pull/28439#discussion_r1058395532
##
airflow/utils/log/colored_log.py:
##
@@ -38,7 +40,7 @@
BOLD_OFF = esc("22")
-class CustomTTYColoredFormatter(TTYColoredFormatter):
+class CustomTTYColoredFormat
vchiapaikeo commented on issue #27166:
URL: https://github.com/apache/airflow/issues/27166#issuecomment-1366700457
Wanted to get started with some light UI work here so gave this simple issue
a shot. Hope that's okay @shashwata27 - I see your PR was a bit stale.
PR: https://github.com
vchiapaikeo opened a new pull request, #28624:
URL: https://github.com/apache/airflow/pull/28624
closes: [#ISSUE](https://github.com/apache/airflow/issues/27166)
Saw that https://github.com/apache/airflow/pull/27195/files might be stale
and it was simple enough so I went ahead
ephraimbuddy commented on code in PR #28619:
URL: https://github.com/apache/airflow/pull/28619#discussion_r1058372127
##
airflow/api/client/json_client.py:
##
@@ -24,9 +24,25 @@
class Client(api_client.Client):
-"""Json API client implementation."""
+"""Json API cli
ephraimbuddy commented on code in PR #28569:
URL: https://github.com/apache/airflow/pull/28569#discussion_r1058365263
##
airflow/providers/slack/provider.yaml:
##
@@ -39,7 +39,7 @@ versions:
- 1.0.0
dependencies:
- - apache-airflow>=2.3.0
+ - apache-airflow>=2.6.0.dev0
potiuk commented on code in PR #28569:
URL: https://github.com/apache/airflow/pull/28569#discussion_r1058359807
##
airflow/providers/slack/provider.yaml:
##
@@ -39,7 +39,7 @@ versions:
- 1.0.0
dependencies:
- - apache-airflow>=2.3.0
+ - apache-airflow>=2.6.0.dev0
Review
ephraimbuddy commented on code in PR #28256:
URL: https://github.com/apache/airflow/pull/28256#discussion_r1058346122
##
airflow/dag_processing/manager.py:
##
@@ -777,8 +777,9 @@ def clear_nonexistent_import_errors(self, session):
:param session: session for ORM operati
jbampton opened a new pull request, #28623:
URL: https://github.com/apache/airflow/pull/28623
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
for more in
potiuk commented on code in PR #28617:
URL: https://github.com/apache/airflow/pull/28617#discussion_r1058300269
##
airflow/providers/ssh/operators/ssh.py:
##
@@ -63,7 +63,10 @@ class SSHOperator(BaseOperator):
"""
template_fields: Sequence[str] = ("command", "environ
aydinemre commented on issue #24669:
URL: https://github.com/apache/airflow/issues/24669#issuecomment-1366600132
I am trying to use the KubernetesPodOperator in Apache Airflow to specify
resource requirements for my pods, but I am unable to pass the resources
parameter to the operator. I ha
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 2169b39d7b Migrate Google Cloud hooks test to `pytest` (#28301)
add 76186bb588 Fix code docstrings (#28622)
No ne
potiuk merged PR #28622:
URL: https://github.com/apache/airflow/pull/28622
--
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
boring-cyborg[bot] commented on PR #28622:
URL: https://github.com/apache/airflow/pull/28622#issuecomment-1366599533
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 on to GitHub
Taragolis commented on PR #28502:
URL: https://github.com/apache/airflow/pull/28502#issuecomment-1366570974
@vincbeck need resolve conflicts
--
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
Taragolis commented on PR #28353:
URL: https://github.com/apache/airflow/pull/28353#issuecomment-1366568491
@avicol doc spellcheck 😢
--
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
ephraimbuddy commented on code in PR #28481:
URL: https://github.com/apache/airflow/pull/28481#discussion_r1058266047
##
airflow/models/dagwarning.py:
##
@@ -72,6 +73,11 @@ def purge_inactive_dag_warnings(cls, session: Session =
NEW_SESSION) -> None:
:return: None
Taragolis commented on PR #28590:
URL: https://github.com/apache/airflow/pull/28590#issuecomment-1366560903
@BobDu I think this changes conflicts with
[pytest-capture-warnings](https://github.com/athinkingape/pytest-capture-warnings)
plugin. I check locally and with our current settings and
ReneTC commented on issue #28603:
URL: https://github.com/apache/airflow/issues/28603#issuecomment-1366555038
I see. I am using
Composer version:
2.0.28
Airflow version:
2.3.3
My providers settings are set to:
` apache-airflow-providers-sftp = "~=2.5.0"`
ephraimbuddy commented on issue #28499:
URL: https://github.com/apache/airflow/issues/28499#issuecomment-1366545805
I think we should do away with this warning
--
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
Taragolis commented on issue #28603:
URL: https://github.com/apache/airflow/issues/28603#issuecomment-1366540261
@ReneTC Could you provide a bit more information about which version of
providers are you use?
Did you tried latest version of SFTP provider? Because I see in [provider
ch
Taragolis commented on code in PR #28569:
URL: https://github.com/apache/airflow/pull/28569#discussion_r1058237022
##
airflow/providers/slack/provider.yaml:
##
@@ -39,7 +39,7 @@ versions:
- 1.0.0
dependencies:
- - apache-airflow>=2.3.0
+ - apache-airflow>=2.6.0.dev0
Rev
potiuk commented on issue #27952:
URL: https://github.com/apache/airflow/issues/27952#issuecomment-1366522100
Just wondering if we can somehow add a precommit to prevent such cases (and
scan all our code to search for similar cases) 🤔
--
This is an automated message from the Apache Git S
potiuk commented on issue #28280:
URL: https://github.com/apache/airflow/issues/28280#issuecomment-1366520133
> Unfortunately we would technically break backward compatibility if we do
this, but I don’t think there’s a good way around it. Maybe alternatively we
should introduce a config to
ephraimbuddy commented on code in PR #28569:
URL: https://github.com/apache/airflow/pull/28569#discussion_r1058190702
##
airflow/providers/slack/provider.yaml:
##
@@ -39,7 +39,7 @@ versions:
- 1.0.0
dependencies:
- - apache-airflow>=2.3.0
+ - apache-airflow>=2.6.0.dev0
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 0e349d80bb Add FTPSFileTransmitOperator (#28318)
add 2169b39d7b Migrate Google Cloud hooks test to `pytest` (#2
Taragolis merged PR #28301:
URL: https://github.com/apache/airflow/pull/28301
--
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
stamixthereal opened a new pull request, #28622:
URL: https://github.com/apache/airflow/pull/28622
Fix docstrings grammar
--
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
bdsoha closed pull request #28621: Extracted workspace cleanup to a CI action
URL: https://github.com/apache/airflow/pull/28621
--
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.
stamixthereal closed pull request #28607: Minor typos and doc-strings added
URL: https://github.com/apache/airflow/pull/28607
--
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
bdsoha opened a new pull request, #28621:
URL: https://github.com/apache/airflow/pull/28621
Extracted the workspace cleanup step to a dedicated CI action.
--
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 t
boring-cyborg[bot] commented on PR #28620:
URL: https://github.com/apache/airflow/pull/28620#issuecomment-1366479223
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
maxnathaniel opened a new pull request, #28620:
URL: https://github.com/apache/airflow/pull/28620
Made some minor changes to the Scheduler documentation. One is a typo, and
others are changes to make the sentences less awkward.
---
**^ Add meaningful description above**
dstandish closed pull request #28616: Reduce logging level of `gunicorn.access`
to warning
URL: https://github.com/apache/airflow/pull/28616
--
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
dstandish commented on PR #28616:
URL: https://github.com/apache/airflow/pull/28616#issuecomment-1366477472
Ah interesting ... thanks for that. Now the behavior makes a lot more
sense. Setting to empty string works for me.
--
This is an automated message from the Apache Git Service.
To
stamixthereal opened a new pull request, #28619:
URL: https://github.com/apache/airflow/pull/28619
Code readability improvements in Client class and methods in json_client
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 0fae6a0c9e Add table_resource to template fields for
BigQueryCreateEmptyTableOperator (#28235)
add 0e349d80bb
Taragolis merged PR #28318:
URL: https://github.com/apache/airflow/pull/28318
--
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
This is an automated email from the ASF dual-hosted git repository.
taragolis pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 467a5e3ab2 Fix calendar view for CronTriggerTimeTable dags (#28411)
add 0fae6a0c9e Add table_resource to templa
Taragolis merged PR #28235:
URL: https://github.com/apache/airflow/pull/28235
--
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
boring-cyborg[bot] commented on PR #28235:
URL: https://github.com/apache/airflow/pull/28235#issuecomment-1366455750
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 on to GitHub
uranusjr commented on PR #28616:
URL: https://github.com/apache/airflow/pull/28616#issuecomment-1366454296
I took a look at the surrounding code and it seems that access logs are
currently explicitly enabled by whoever implemented it in the first place.
Gunicorn does not emit access logs un
bkossakowska opened a new pull request, #28618:
URL: https://github.com/apache/airflow/pull/28618
Add deferrable mode to DataprocInstantiateWorkflowTemplateOperator
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apa
101 - 200 of 200 matches
Mail list logo