rino0601 opened a new issue, #25271:
URL: https://github.com/apache/airflow/issues/25271
### Apache Airflow version
2.3.3 (latest released)
### What happened
In 2.3.3
If I use
https://airflow.apache.org/docs/apache-airflow/stable/plugins.html#plugins-as-python-pac
boring-cyborg[bot] commented on issue #25271:
URL: https://github.com/apache/airflow/issues/25271#issuecomment-1193667770
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
Jorricks commented on PR #23402:
URL: https://github.com/apache/airflow/pull/23402#issuecomment-1193669790
Hey,
Sorry I dropped the ball on this one. Had some other projects going on.
Rebased and fixed conflict :)
--
This is an automated message from the Apache Git Service.
To respon
potiuk opened a new pull request, #25272:
URL: https://github.com/apache/airflow/pull/25272
When we moved www asset compilation from container to the host in #25169,
we removed yarn from the image and the old instructions are no longer
valid. Instead of in-container yarn dev we expect n
jplauri commented on PR #19596:
URL: https://github.com/apache/airflow/pull/19596#issuecomment-1193739796
Just to avoid this going stale as I think it could be helpful. @dstandish do
you think some work is still required?
--
This is an automated message from the Apache Git Service.
To res
potiuk merged PR #25270:
URL: https://github.com/apache/airflow/pull/25270
--
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 bb5a8ec88a Add werkzeug limitation until flask-logi
potiuk commented on PR #25270:
URL: https://github.com/apache/airflow/pull/25270#issuecomment-1193748703
Thanks @rino0601 ! - I was just looking at some early warnings from some of
our PRs failing, resulting from this error !
--
This is an automated message from the Apache Git Service.
T
argibbs commented on issue #15596:
URL: https://github.com/apache/airflow/issues/15596#issuecomment-1193751812
#25147 improves things slightly, but add enough SLAs to your system and you
hit another issue.
Re-opening while I work on a second MR.
--
This is an automated mess
argibbs opened a new issue, #15596:
URL: https://github.com/apache/airflow/issues/15596
**Apache Airflow version**: 2.0.1 and 2.0.2
**Kubernetes version (if you are using kubernetes)** (use `kubectl
version`): N/A
**Environment**: Celery executors, Redis + Postgres
- **C
potiuk commented on PR #25270:
URL: https://github.com/apache/airflow/pull/25270#issuecomment-1193758570
I also commented in the issue
https://github.com/maxcountryman/flask-login/issues/686#issuecomment-1193755779
- this is I thin 3rd time over 3 years that we experienced some Werkzeug +
uranusjr commented on code in PR #25260:
URL: https://github.com/apache/airflow/pull/25260#discussion_r928616540
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -590,7 +590,7 @@ def __init__(
python_params: Optional[List[str]] = None,
jar_params:
potiuk commented on issue #25271:
URL: https://github.com/apache/airflow/issues/25271#issuecomment-1193761640
Yeah. It would be great to get to the bottom of this. Happy to help with
investigation if you have some more findings.
--
This is an automated message from the Apache Git Service.
boring-cyborg[bot] commented on issue #25274:
URL: https://github.com/apache/airflow/issues/25274#issuecomment-1193763743
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
vlcheong opened a new issue, #25274:
URL: https://github.com/apache/airflow/issues/25274
### Apache Airflow version
2.3.3 (latest released)
### What happened
I trying to make SqlSensor to work with Oracle database, I've installed all
the required provider and successfull
alexott commented on code in PR #25260:
URL: https://github.com/apache/airflow/pull/25260#discussion_r928625968
##
airflow/providers/databricks/operators/databricks.py:
##
@@ -590,7 +590,7 @@ def __init__(
python_params: Optional[List[str]] = None,
jar_params:
uranusjr closed issue #13158: UI Doesn't use all of Bootstrap theme css,
Airflow 2.0
URL: https://github.com/apache/airflow/issues/13158
--
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
uranusjr merged PR #15277:
URL: https://github.com/apache/airflow/pull/15277
--
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
This is an automated email from the ASF dual-hosted git repository.
uranusjr 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 f634361cb7 Remove support for FAB `APP_THEME` (#1
anja-istenic opened a new pull request, #25277:
URL: https://github.com/apache/airflow/pull/25277
As my bugfix in this PR: https://github.com/apache/airflow/pull/25227 has
been fixed by another merge, I suggest we still add a test for this issue so as
to not introduce a similar bug in the f
bbovenzi commented on code in PR #25272:
URL: https://github.com/apache/airflow/pull/25272#discussion_r928645232
##
dev/breeze/src/airflow_breeze/commands/developer_commands.py:
##
@@ -558,13 +566,20 @@ def static_checks(
allow_extra_args=True,
),
)
+@click.option
anja-istenic commented on PR #25227:
URL: https://github.com/apache/airflow/pull/25227#issuecomment-1193791308
Thanks both for your comments. I can confirm that PR
https://github.com/apache/airflow/pull/23971 fixes the issue I addressed with
this PR. I would, however, suggest we still add t
anja-istenic commented on PR #25277:
URL: https://github.com/apache/airflow/pull/25277#issuecomment-1193793884
@kazanzhy @potiuk Added an extra test for the exasol hook, as mentioned in
https://github.com/apache/airflow/pull/25227.
--
This is an automated message from the Apache Git Servi
bbovenzi commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1193802421
> You are right @ephraimbuddy , was trying to add something that Brent
needed but there must be a better way
A few options:
1. The UI is just inefficient and has to do mul
chenglongyan closed pull request #25261: Delete redundant system test
bigquery_to_bigquery
URL: https://github.com/apache/airflow/pull/25261
--
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
bbovenzi commented on PR #25258:
URL: https://github.com/apache/airflow/pull/25258#issuecomment-1193811426
Good idea. That is a lot more consistent from dag -> run -> task. My initial
reactions:
- I know I had a header in the DAG details, but seeing it for a run and task
instance. It
bbovenzi commented on code in PR #25258:
URL: https://github.com/apache/airflow/pull/25258#discussion_r928665094
##
airflow/www/static/js/dag/details/dagRun/index.tsx:
##
@@ -23,7 +23,7 @@ import {
Box,
Button,
Link,
- Divider,
+ Divider, Table, Tbody, Tr, Td, Heading
bbovenzi commented on code in PR #25258:
URL: https://github.com/apache/airflow/pull/25258#discussion_r928666082
##
airflow/www/static/js/dag/details/dagRun/index.tsx:
##
@@ -74,85 +74,100 @@ const DagRun = ({ runId }: Props) => {
const detailsLink = appendSearchParams(dagRun
bbovenzi commented on code in PR #25258:
URL: https://github.com/apache/airflow/pull/25258#discussion_r928666904
##
airflow/www/static/js/dag/details/taskInstance/Details.tsx:
##
@@ -21,7 +21,7 @@ import React from 'react';
import {
Text,
Box,
- Flex,
+ Flex, Table, Tbo
vishalyadav93 opened a new issue, #25278:
URL: https://github.com/apache/airflow/issues/25278
### Apache Airflow version
2.3.3 (latest released)
### What happened
In Tree view of airflow, manual run is always showing after schedule run
even though manual run is before sc
boring-cyborg[bot] commented on issue #25278:
URL: https://github.com/apache/airflow/issues/25278#issuecomment-1193816361
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
Conformist101 opened a new pull request, #25279:
URL: https://github.com/apache/airflow/pull/25279
Added Europcar to the INTHEWILD.md file
--
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 spe
boring-cyborg[bot] commented on PR #25279:
URL: https://github.com/apache/airflow/pull/25279#issuecomment-1193821361
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
Taragolis closed pull request #24246: Make amazon athena, batch, lambda, and
eks operators more consistent
URL: https://github.com/apache/airflow/pull/24246
--
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
Taragolis commented on PR #24246:
URL: https://github.com/apache/airflow/pull/24246#issuecomment-1193841630
Close for now, plan back to this PR after changes in Hook(s)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
potiuk commented on issue #23361:
URL: https://github.com/apache/airflow/issues/23361#issuecomment-1193845550
> The behavior was not changed after the above issue was filed, but the
following warning was added to the SQLAlchemy documentation:
Ah. That Would indeed explain it.
alexkruc opened a new pull request, #25280:
URL: https://github.com/apache/airflow/pull/25280
Based on the discussion on #24728, this is the PR to allow the users to add
their own hyperlinks to the `owner` property.
The idea behind this PR is as follows -
Similar to what we do wit
alexkruc commented on PR #25280:
URL: https://github.com/apache/airflow/pull/25280#issuecomment-1193850381
Now that I'm thinking of this, maybe I should rename the `dag_owner_links`
table to something like `dag_owner_attributes` -> it sounds more generic if we
would like to add more owner a
potiuk commented on issue #23361:
URL: https://github.com/apache/airflow/issues/23361#issuecomment-1193861793
> I do not actually know about is it supported by other DB engine and is it
has exactly the same behaviour.
@Taragolis woudl be worth checking. The DagRun lock `SELECT FOR UP
potiuk closed issue #25269: ImportError: cannot import name 'parse_rule' from
'werkzeug.routing'
(/root/airflow-venv/airflow_virtualenv/lib/python3.7/site-packages/werkzeug/routing/__init__.py)
URL: https://github.com/apache/airflow/issues/25269
--
This is an automated message from the Apach
potiuk commented on issue #25269:
URL: https://github.com/apache/airflow/issues/25269#issuecomment-1193865158
Closed by #25270
--
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 commen
potiuk commented on issue #25232:
URL: https://github.com/apache/airflow/issues/25232#issuecomment-1193882569
> Hi, I will provide fix for that issue.
Cool!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
potiuk commented on PR #25239:
URL: https://github.com/apache/airflow/pull/25239#issuecomment-1193898788
It looks great and I am happy to merge, but I have one thought about another
contributing factor that we can include.
We are pretty rigorously follow "linking issues to PRs". There
potiuk commented on issue #25274:
URL: https://github.com/apache/airflow/issues/25274#issuecomment-1193911726
Please use the "Common-sql" sensor for that. Instead of `from
airflow.sensors import SqlSensor` use `from
airflow.providers.common.sql.sensors import SqlSensors` - that should fix t
potiuk commented on issue #25274:
URL: https://github.com/apache/airflow/issues/25274#issuecomment-1193912788
Mybe I will find another solution for that in the future as well - I will
think and dig a bit deeper.
--
This is an automated message from the Apache Git Service.
To respond to th
bbovenzi commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928763282
##
airflow/api_connexion/openapi/v1.yaml:
##
@@ -3228,6 +3228,9 @@ components:
owner:
type: string
readOnly: true
+outlets_data
bbovenzi commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928763692
##
airflow/www/static/js/types/api-generated.ts:
##
@@ -1327,6 +1327,7 @@ export interface components {
class_ref?: components["schemas"]["ClassReference"];
bbovenzi commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928764133
##
tests/api_connexion/endpoints/test_task_endpoint.py:
##
@@ -332,6 +336,7 @@ def test_get_tasks_mapped(self):
"execution_timeout": None,
bbovenzi commented on PR #25249:
URL: https://github.com/apache/airflow/pull/25249#issuecomment-1193915672
Yes, this would work for my usecase, but not including the dataset id isn't
ideal for a public API. Also, this pattern is quite different from what we're
discussing in https://github.c
bbovenzi commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928764133
##
tests/api_connexion/endpoints/test_task_endpoint.py:
##
@@ -332,6 +336,7 @@ def test_get_tasks_mapped(self):
"execution_timeout": None,
potiuk commented on PR #25267:
URL: https://github.com/apache/airflow/pull/25267#issuecomment-1193919634
Rebased to account for the "Werkzeug" fix.
--
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
potiuk commented on PR #25277:
URL: https://github.com/apache/airflow/pull/25277#issuecomment-1193923037
Cool!
--
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 unsubscrib
potiuk commented on PR #25277:
URL: https://github.com/apache/airflow/pull/25277#issuecomment-1193923541
I rebased it as we had some breaking main, so just in case.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
potiuk commented on PR #25261:
URL: https://github.com/apache/airflow/pull/25261#issuecomment-1193924564
Rebased to account for latest "main" breakage
--
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 g
potiuk merged PR #25279:
URL: https://github.com/apache/airflow/pull/25279
--
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 #25279:
URL: https://github.com/apache/airflow/pull/25279#issuecomment-1193925625
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 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 a5cbcb5677 add europcar to list of companies using
hansmbakker opened a new issue, #25281:
URL: https://github.com/apache/airflow/issues/25281
### Apache Airflow version
2.3.3 (latest released)
### What happened
The `git clone` command failed because of files in
[/tests/www/test_logs/dag_for_testing_log_view](https://git
boring-cyborg[bot] commented on issue #25281:
URL: https://github.com/apache/airflow/issues/25281#issuecomment-1193937915
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
mk12306 opened a new issue, #25282:
URL: https://github.com/apache/airflow/issues/25282
### Apache Airflow version
2.1.4
### What happened
psycopg2.errors.UndefinedColumn: column dag.concurrency does not exist
### What you think should happen instead
enter t
boring-cyborg[bot] commented on issue #25282:
URL: https://github.com/apache/airflow/issues/25282#issuecomment-1193938162
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
Aakcht opened a new pull request, #25283:
URL: https://github.com/apache/airflow/pull/25283
This PR allows users to specify custom labels for secrets/configmaps created
with `extraSecrets`/`extraConfigMaps` params
---
**^ Add meaningful description above**
Read the *
pierrejeambrun commented on PR #25272:
URL: https://github.com/apache/airflow/pull/25272#issuecomment-1193957243
Great
--
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 un
hansmbakker commented on issue #25281:
URL: https://github.com/apache/airflow/issues/25281#issuecomment-1193958936
Workaround: cloning inside WSL works, but I believe that using filenames
containing characters that are unsafe in a path is not a best practice.
--
This is an automated messa
potiuk closed issue #25282: psycopg2.errors.UndefinedColumn: column
dag.concurrency does not exist
URL: https://github.com/apache/airflow/issues/25282
--
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
potiuk commented on PR #25272:
URL: https://github.com/apache/airflow/pull/25272#issuecomment-1193980561
BTW. @bbovenzi @pierrejeambrun - one comment, the nice thing about using
pre-commit is that you do not have to care about maintaining your 'node + yarn'
in the right version, so it wi
boring-cyborg[bot] commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1193987650
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the mess
Lazloo opened a new issue, #25286:
URL: https://github.com/apache/airflow/issues/25286
### Apache Airflow version
2.3.3 (latest released)
### What happened
Unfortunately, I cannot get further information about my actual error
because of the following KeyError
```
potiuk closed issue #25281: Cannot clone Airflow repo
URL: https://github.com/apache/airflow/issues/25281
--
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
dstandish commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1193989243
> > You are right @ephraimbuddy , was trying to add something that Brent
needed but there must be a better way
>
> A few options:
>
> 1. The UI is just inefficient and ha
potiuk commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1193993554
Hey @alexott - maybe you or someone from your team can help ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
alexott commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1193995202
What is the version of the Databricks provider?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
alexott commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1193999419
What is the version of the Databricks provider? Can you also provide an
example of the code - what operator are you using, etc.?
--
This is an automated message from the Apache G
dstandish commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194004358
update: figured out how to just add URI... will do that
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
Lazloo commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194004837
Here the code that run the deployment:
```
from airflow import AirflowException
from airflow.models import Variable
from airflow.providers.databricks.operators.databri
bbovenzi commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194006553
> @bbovenzi what's the context again? TI (i.e. downstream) or DAG (i.e.
upstream)? Either way we know, or could know, the datasets that it maps to, and
could provide a dictionary from
Lazloo commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194008577
Regarding the Deployment. I currently do not find the concrete version but
it is a legacy version (**not** E2)
--
This is an automated message from the Apache Git Service.
To resp
alexott commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194010109
what is the version of `apache-airflow-providers-databricks` ? latest?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
Lazloo commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194018706
it is based on quay.io/astronomer/ap-airflow:2.3.3-onbuild
Does this help?
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
dstandish commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928861034
##
tests/api_connexion/endpoints/test_task_endpoint.py:
##
@@ -332,6 +336,7 @@ def test_get_tasks_mapped(self):
"execution_timeout": None,
dstandish commented on code in PR #25249:
URL: https://github.com/apache/airflow/pull/25249#discussion_r928862024
##
airflow/api_connexion/openapi/v1.yaml:
##
@@ -3228,6 +3228,9 @@ components:
owner:
type: string
readOnly: true
+outlets_dat
Lazloo commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194027044
Before we used quay.io/astronomer/ap-airflow:2.2.5-onbuild. After the update
we got this error
--
This is an automated message from the Apache Git Service.
To respond to the messa
dstandish commented on code in PR #25258:
URL: https://github.com/apache/airflow/pull/25258#discussion_r928871405
##
airflow/www/static/js/dag/details/dagRun/index.tsx:
##
@@ -74,85 +74,100 @@ const DagRun = ({ runId }: Props) => {
const detailsLink = appendSearchParams(dagRu
alexott commented on issue #25286:
URL: https://github.com/apache/airflow/issues/25286#issuecomment-1194043765
it looks like it's caused by this PR:
https://github.com/apache/airflow/pull/21709/files - I'll investigate it
--
This is an automated message from the Apache Git Service.
To res
dstandish commented on PR #25258:
URL: https://github.com/apache/airflow/pull/25258#issuecomment-1194063341
ok @bbovenzi removed header and moved status into table (and updated the
screenshots)
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
dstandish commented on PR #25258:
URL: https://github.com/apache/airflow/pull/25258#issuecomment-1194064435
btw, to get status icon + status label in same line i had to wrap in a
Flex... not sure why but it worked 🤷
--
This is an automated message from the Apache Git Service.
To resp
potiuk commented on issue #24909:
URL: https://github.com/apache/airflow/issues/24909#issuecomment-1194067604
Question: Which version of which database do you have @amoGLingle ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
potiuk commented on PR #25196:
URL: https://github.com/apache/airflow/pull/25196#issuecomment-1194070946
Merging. Random issue with MsSQL
--
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
potiuk merged PR #25196:
URL: https://github.com/apache/airflow/pull/25196
--
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 closed issue #24820: Dag disappears when DAG tag is longer than 100 char
limit
URL: https://github.com/apache/airflow/issues/24820
--
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
boring-cyborg[bot] commented on PR #25196:
URL: https://github.com/apache/airflow/pull/25196#issuecomment-1194071190
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 a5cbcb5677 add europcar to list of companies using Airflow (#25279)
add 4b28635b20 Disallow any dag tags longer th
potiuk commented on PR #25258:
URL: https://github.com/apache/airflow/pull/25258#issuecomment-1194072596
> btw, to get status icon + status label in same line i had to wrap in a
Flex... not sure why but it worked shrug
Probably one of those:
 were added to refs/heads/main by this push:
new d7f4ee1786 Fix ExternalTaskSensor not working with
dstandish commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194079246
i added just URI to dataset event response at top level. i think it's a good
solution. it's reasonable to include the logical key along with the surr key
in a response like this.
ashb commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194083324
For consistency should we call the field `dataset_uri`? (It's longer so I'm
not a huge fan of it, but :shrug:)
--
This is an automated message from the Apache Git Service.
To respond to
dstandish commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194092914
> For consistency should we call the field `dataset_uri`? (It's longer so
I'm not a huge fan of it, but 🤷)
thought about that actually. probably a good idea.
--
This is an a
dstandish commented on PR #25250:
URL: https://github.com/apache/airflow/pull/25250#issuecomment-1194097345
> > For consistency should we call the field `dataset_uri`? (It's longer so
I'm not a huge fan of it, but 🤷)
>
> thought about that actually. probably a good idea.
actual
1 - 100 of 200 matches
Mail list logo
