norm commented on a change in pull request #22341:
URL: https://github.com/apache/airflow/pull/22341#discussion_r837073475
##
File path: tests/api_connexion/endpoints/test_task_instance_endpoint.py
##
@@ -376,6 +376,16 @@ def test_should_return_404_for_mapped_endpoint(self,
ashb commented on a change in pull request #22123:
URL: https://github.com/apache/airflow/pull/22123#discussion_r837064176
##
File path: airflow/www/static/js/tree/ResetRoot.jsx
##
@@ -0,0 +1,55 @@
+/*!
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more
boring-cyborg[bot] commented on pull request #22580:
URL: https://github.com/apache/airflow/pull/22580#issuecomment-1081411416
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
nitinmahawadiwar opened a new pull request #22580:
URL: https://github.com/apache/airflow/pull/22580
Returning description field for GET a)variables & b) variable API.
SWAGGER documentation updated.
Test cases are incomplete, not able to understand the overall flow for test
dstandish commented on a change in pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#discussion_r837058224
##
File path: airflow/providers_manager.py
##
@@ -138,7 +139,57 @@ def _check_builtin_provider_prefix(provider_package: str,
class_name: str) -> bo
uranusjr commented on issue #10451:
URL: https://github.com/apache/airflow/issues/10451#issuecomment-1081384642
The subclassing solution is not a hack Rendering `.json` with Jinja2 is a
feature by design on the oeprator, and if you don’t want that feature, you
should make your own
dstandish commented on issue #15990:
URL: https://github.com/apache/airflow/issues/15990#issuecomment-1081364324
> @dstandish was the original issue resolved? In this case, please, provide
corresponded note in the changelog. We are still using custom logging. In
another case, please, open
rsibanez89 commented on issue #10451:
URL: https://github.com/apache/airflow/issues/10451#issuecomment-1081354542
> Airflow tried to renders all the values passed to template_fields. In your
case as your are using `EmrAddStepsOperator`, it's template_fields are
`['job_flow_id',
github-actions[bot] commented on pull request #22577:
URL: https://github.com/apache/airflow/pull/22577#issuecomment-1081343157
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
uranusjr commented on pull request #22577:
URL: https://github.com/apache/airflow/pull/22577#issuecomment-1081342662
Technically we don’t have another revision, but explicitness is better.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
uranusjr commented on a change in pull request #22578:
URL: https://github.com/apache/airflow/pull/22578#discussion_r837005669
##
File path: airflow/www/static/js/dag.js
##
@@ -54,6 +54,9 @@ let executionDate = '';
let subdagId = '';
let dagRunId = '';
let mapIndex;
+let
uranusjr commented on a change in pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#discussion_r837002029
##
File path: airflow/providers_manager.py
##
@@ -138,7 +139,57 @@ def _check_builtin_provider_prefix(provider_package: str,
class_name: str) -> bo
egS27 edited a comment on issue #15990:
URL: https://github.com/apache/airflow/issues/15990#issuecomment-1081328848
@dstandish was the original issue resolved? In this case, please, provide
corresponded note in the changelog. We are still using custom logging. In
another case, please,
egS27 commented on issue #15990:
URL: https://github.com/apache/airflow/issues/15990#issuecomment-1081328848
@dstandish was the original issue resolved? In this case, please, provide
corresponded note in the changelog. We are still using custom logging.
--
This is an automated message
dstandish commented on pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#issuecomment-1081328480
> NIT: I'd use Enum or bool for providers/sources, but having string is nice
for debugging too, so no problem wiht it (and TIL about _post_init__)
yeah thought about
atishbits commented on pull request #21793:
URL: https://github.com/apache/airflow/pull/21793#issuecomment-1081309830
> > @ashb this is very useful. Worth considering adding this to all v2.0+
releases rather than just the upcoming release ?
>
> BTW. I recommend you to upgrade to
shuhoy commented on pull request #22506:
URL: https://github.com/apache/airflow/pull/22506#issuecomment-1081296373
Oh! That's exciting!!
I'm grad to contribute to Airflow community!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
github-actions[bot] commented on issue #13692:
URL: https://github.com/apache/airflow/issues/13692#issuecomment-1081269892
This issue has been automatically marked as stale because it has been open
for 30 days with no response from the author. It will be closed in next 7 days
if no
github-actions[bot] commented on pull request #21515:
URL: https://github.com/apache/airflow/pull/21515#issuecomment-1081269826
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
github-actions[bot] closed pull request #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:
lrbsantos commented on issue #19104:
URL: https://github.com/apache/airflow/issues/19104#issuecomment-1081266736
> Hi, @lrbsantos Currently I am using SQLite as the backend as I got other
issues when working with MySQL. But I tried setting
`explicit_defaults_for_timestamp` key set as ON
yuqian90 commented on issue #22381:
URL: https://github.com/apache/airflow/issues/22381#issuecomment-1081260212
@eladkal `max_tries` is an attribute of `TaskInstance`. #21018 is basically
fixing a bug of the web UI to not reset its value by mistake when someone opens
the Task instance web
ferruzzi commented on pull request #22517:
URL: https://github.com/apache/airflow/pull/22517#issuecomment-1081236759
Looks like chain() doesn't like @task decorated methods. Posted to the
Slack server to discuss fix:
https://apache-airflow.slack.com/archives/CCPRP7943/p1648498867915099
potiuk commented on pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#issuecomment-1081227228
(Remnant from time when ProviderInfo was just a tuple it seems).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk commented on pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#issuecomment-1081225929
Also 'airflow providers list' cmd needs fixing too :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
github-actions[bot] commented on pull request #22579:
URL: https://github.com/apache/airflow/pull/22579#issuecomment-1081225362
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
jedcunningham commented on a change in pull request #22573:
URL: https://github.com/apache/airflow/pull/22573#discussion_r836909803
##
File path: setup.py
##
@@ -1033,17 +1050,29 @@ def
replace_extra_requirement_with_provider_packages(extra: str, providers: List
dstandish opened a new pull request #22579:
URL: https://github.com/apache/airflow/pull/22579
When we are running airflow locally with providers installed from sources,
often many providers will be discovered which we haven't installed the deps
for. This generally results in a very large
bbovenzi closed pull request #22123:
URL: https://github.com/apache/airflow/pull/22123
--
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:
bbovenzi opened a new pull request #22123:
URL: https://github.com/apache/airflow/pull/22123
Add a details drawer to the right side of the grid view to quickly see
details and actions for the DAG / Dag Run / Task Instance. This replaces the
modal when clicking on a dag run or task
This is an automated email from the ASF dual-hosted git repository.
bbovenzi pushed a change to branch map-task-modal-btns
in repository https://gitbox.apache.org/repos/asf/airflow.git.
at d4fc739 Update mapped task instance modal buttons
No new revisions were added by this update.
bbovenzi opened a new pull request #22578:
URL: https://github.com/apache/airflow/pull/22578
Right now in graph view, a user can click on a mapped task and see a summary
in a modal. But they cannot see an individual mapped task. This PR fixes that
with:
- Adds a new "All Mapped
dacort commented on issue #20215:
URL: https://github.com/apache/airflow/issues/20215#issuecomment-1081184838
I just published some information about how to use (a custom, while in
preview) boto3 to access the EMR Serverless API.
blag opened a new pull request #22577:
URL: https://github.com/apache/airflow/pull/22577
Update help text for `airflow db upgrade/downgrade` to specify _Alembic_
revision.
---
**^ Add meaningful description above**
Read the **[Pull Request
potiuk edited a comment on pull request #22539:
URL: https://github.com/apache/airflow/pull/22539#issuecomment-1081181094
Just to add more context- unfortunately there is no easy way to to determine
if we are in "dev" (editable) environment or not. We could simply check if we
are in
potiuk edited a comment on pull request #22539:
URL: https://github.com/apache/airflow/pull/22539#issuecomment-1081181094
Just to add more context- unfortunately there is no easy way to to determine
if we are in "dev" (editable) environment or not. We could simply check if we
are in
potiuk commented on pull request #22539:
URL: https://github.com/apache/airflow/pull/22539#issuecomment-1081181094
Just to add more context- unfortunately there is no easy way to to determine
if we are in "dev" (editable) environment or not. We could simply check if we
are in "BREEZE"
potiuk commented on pull request #22539:
URL: https://github.com/apache/airflow/pull/22539#issuecomment-1081175189
> @potiuk i looked at that method, it doesn't actually import the provider
there, but just stores in `_provider_dict`.
>
> are you thinking we add more information for
dstandish commented on pull request #22539:
URL: https://github.com/apache/airflow/pull/22539#issuecomment-1081174022
@potiuk i looked at that method, it doesn't actually import the provider
there, but just stores in `_provider_dict`.
are you thinking we add more information for
potiuk commented on pull request #22506:
URL: https://github.com/apache/airflow/pull/22506#issuecomment-1081158229
CONGRATS!
--
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
jedcunningham commented on pull request #22506:
URL: https://github.com/apache/airflow/pull/22506#issuecomment-1081127499
@shuhoy, congrats on your first commit! In fact, you are the 2000th
contributor to Airflow !
