[jira] [Commented] (AIRFLOW-5071) Thousand os Executor reports task instance X finished (success) although the task says its queued. Was the task killed externally?

2021-03-05 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17296472#comment-17296472 ] ASF GitHub Bot commented on AIRFLOW-5071: - rsingh2021 commented on issue #10790: URL:

[GitHub] [airflow] rsingh2021 commented on issue #10790: Copy of [AIRFLOW-5071] JIRA: Thousands of Executor reports task instance X finished (success) although the task says its queued. Was the task k

2021-03-05 Thread GitBox
rsingh2021 commented on issue #10790: URL: https://github.com/apache/airflow/issues/10790#issuecomment-791887378 @turbaszek - Is this error is fixed in 2.0 ? This is an automated message from the Apache Git Service. To

[GitHub] [airflow] jyotidhiman0610 commented on issue #14543: string templated fields are not rendered literally

2021-03-05 Thread GitBox
jyotidhiman0610 commented on issue #14543: URL: https://github.com/apache/airflow/issues/14543#issuecomment-791879598 Let me know if you are working on this @YangMuye, if not I can take this up. This is an automated message

[GitHub] [airflow] jmcarp commented on a change in pull request #14589: Drop third-party lazy proxy.

2021-03-05 Thread GitBox
jmcarp commented on a change in pull request #14589: URL: https://github.com/apache/airflow/pull/14589#discussion_r588834321 ## File path: airflow/utils/proxy.py ## @@ -0,0 +1,47 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[GitHub] [airflow] markhopson commented on pull request #14263: Added retry to ECS Operator

2021-03-05 Thread GitBox
markhopson commented on pull request #14263: URL: https://github.com/apache/airflow/pull/14263#issuecomment-791841984 @ashb Friendly bump here This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] github-actions[bot] commented on pull request #14400: Add conf not importable from airflow rule

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14400: URL: https://github.com/apache/airflow/pull/14400#issuecomment-791839169 [The Workflow run](https://github.com/apache/airflow/actions/runs/626256077) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] mik-laj commented on issue #14623: Passing a jinja template in TriggerDagRunOperator conf raises json.decoder.JSONDecodeError

2021-03-05 Thread GitBox
mik-laj commented on issue #14623: URL: https://github.com/apache/airflow/issues/14623#issuecomment-791838484 Here is workaround: https://github.com/apache/airflow/issues/11983 This is an automated message from the Apache

[GitHub] [airflow] ephraimbuddy commented on issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

2021-03-05 Thread GitBox
ephraimbuddy commented on issue #14635: URL: https://github.com/apache/airflow/issues/14635#issuecomment-791831776 @zachliu, did you try using `update_mask=is_paused` instead of `update_mask=['is_paused']`? This is an

[GitHub] [airflow] zachliu commented on issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

2021-03-05 Thread GitBox
zachliu commented on issue #14635: URL: https://github.com/apache/airflow/issues/14635#issuecomment-791825913 btw, this would be more of an issue once we start using the generated client for example, in Python client, this example in the doc won't work: ```python dag = DAG(

[GitHub] [airflow] zachliu commented on issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

2021-03-05 Thread GitBox
zachliu commented on issue #14635: URL: https://github.com/apache/airflow/issues/14635#issuecomment-791824195 sorry, that's a typo i followed the example This is an automated message from the Apache Git Service. To

[GitHub] [airflow] kaxil opened a new pull request #14638: Cache Hook when initializing `CloudFormationCreateStackSensor`

2021-03-05 Thread GitBox
kaxil opened a new pull request #14638: URL: https://github.com/apache/airflow/pull/14638 - `CloudFormationCreateStackSensor` created `AWSCloudFormationHook` in its init method - This commit delays creation of hook until it is needed --- **^ Add meaningful description

[GitHub] [airflow] github-actions[bot] closed pull request #11410: Optionally use iam role for redshift unload to s3.

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11410: URL: https://github.com/apache/airflow/pull/11410 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #11437: Update pod-template-file.yaml

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11437: URL: https://github.com/apache/airflow/pull/11437 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #11540: Support DATE and DATETIME in MSSQLToGCSOperator

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11540: URL: https://github.com/apache/airflow/pull/11540 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] commented on pull request #11392: Change some warning logs to info/debug

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #11392: URL: https://github.com/apache/airflow/pull/11392#issuecomment-791813250 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] [airflow] github-actions[bot] closed pull request #11843: Fix issue with moving end_date in elastic dag

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11843: URL: https://github.com/apache/airflow/pull/11843 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #11699: Enforce ESLint (JavaScript linting) with pre-commit hooks

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11699: URL: https://github.com/apache/airflow/pull/11699 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #11627: Add D205 pydocstyle check

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #11627: URL: https://github.com/apache/airflow/pull/11627 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #12376: Use new syntax for xcom pull in example dags

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #12376: URL: https://github.com/apache/airflow/pull/12376 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #12355: add ability to import dags in a subfolder with a zip file

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #12355: URL: https://github.com/apache/airflow/pull/12355 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #12271: Update example to use dag decorator tag

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #12271: URL: https://github.com/apache/airflow/pull/12271 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] github-actions[bot] closed pull request #12599: Restrict "build images" workflow to certain branches

2021-03-05 Thread GitBox
github-actions[bot] closed pull request #12599: URL: https://github.com/apache/airflow/pull/12599 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [airflow] kaxil merged pull request #14606: Use built-in `cached_property` on Python 3.8 where possible

2021-03-05 Thread GitBox
kaxil merged pull request #14606: URL: https://github.com/apache/airflow/pull/14606 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

[airflow] branch master updated (e468b31 -> e7bb17a)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from e468b31 Fix type hints in OpsgenieAlertOperator (#14637) add e7bb17a Use built-in `cached_property` on

[airflow] branch master updated (779ecc4 -> e468b31)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 779ecc4 Replace Python 2.7 with 3.6 in IMAGES.rst (#14636) add e468b31 Fix type hints in

[GitHub] [airflow] kaxil merged pull request #14637: Fix type hints in OpsgenieAlertOperator

2021-03-05 Thread GitBox
kaxil merged pull request #14637: URL: https://github.com/apache/airflow/pull/14637 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

[GitHub] [airflow] github-actions[bot] commented on pull request #14637: Fix type hints in OpsgenieAlertOperator

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14637: URL: https://github.com/apache/airflow/pull/14637#issuecomment-791805256 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] fjmacagno commented on issue #14613: Better "dependency already registered" warning message

2021-03-05 Thread GitBox
fjmacagno commented on issue #14613: URL: https://github.com/apache/airflow/issues/14613#issuecomment-791805189 2.0.1 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [airflow] mik-laj commented on issue #14613: Better "dependency already registered" warning message

2021-03-05 Thread GitBox
mik-laj commented on issue #14613: URL: https://github.com/apache/airflow/issues/14613#issuecomment-791805009 what version of Airflow do you use? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] kaxil opened a new pull request #14637: Fix type hints in OpsgenieAlertOperator

2021-03-05 Thread GitBox
kaxil opened a new pull request #14637: URL: https://github.com/apache/airflow/pull/14637 `actions` and `tags` are list of strings. The docstrings contain the correct type but for some reason TypeHints had it wrong. Docs: https://docs.opsgenie.com/docs/alert-api#section-create-alert

[GitHub] [airflow] mik-laj commented on issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

2021-03-05 Thread GitBox
mik-laj commented on issue #14635: URL: https://github.com/apache/airflow/issues/14635#issuecomment-791801688 From what I can see, you are not correctly defining the update mask. See:

[GitHub] [airflow] arkadiuszbach edited a comment on issue #10860: Timeouts in Airflow scheduler logs on AKS

2021-03-05 Thread GitBox
arkadiuszbach edited a comment on issue #10860: URL: https://github.com/apache/airflow/issues/10860#issuecomment-791796674 On Azure LoadBalancers have tcp idle timeout set as default to 4 minutes(it is visible in JSON View), so if you connect and don't interact with it in more than 4

[airflow] branch master updated (b0d6069 -> 779ecc4)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from b0d6069 Fix broken static check on Master (#14633) add 779ecc4 Replace Python 2.7 with 3.6 in IMAGES.rst

[GitHub] [airflow] kaxil merged pull request #14636: Replace Python 2.7 with 3.6 in IMAGES.rst

2021-03-05 Thread GitBox
kaxil merged pull request #14636: URL: https://github.com/apache/airflow/pull/14636 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

[GitHub] [airflow] github-actions[bot] commented on pull request #14633: Fix broken static check on Master

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14633: URL: https://github.com/apache/airflow/pull/14633#issuecomment-791797900 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

[airflow] branch master updated (201d6e4 -> b0d6069)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 201d6e4 Remove unused usage of logging module (#14632) add b0d6069 Fix broken static check on Master

[GitHub] [airflow] kaxil merged pull request #14633: Fix broken static check on Master

2021-03-05 Thread GitBox
kaxil merged pull request #14633: URL: https://github.com/apache/airflow/pull/14633 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

[GitHub] [airflow] kaxil merged pull request #14632: Remove unused usage of logging module

2021-03-05 Thread GitBox
kaxil merged pull request #14632: URL: https://github.com/apache/airflow/pull/14632 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

[airflow] branch master updated: Remove unused usage of logging module (#14632)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 201d6e4 Remove unused usage of logging

[airflow] branch master updated (4589375 -> 1bf76f9)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 4589375 Fix various links in README.md (#14630) add 1bf76f9 Fix broken link in LOCAL_VIRTUALENV.rst

[GitHub] [airflow] kaxil merged pull request #14634: Fix broken link in LOCAL_VIRTUALENV.rst

2021-03-05 Thread GitBox
kaxil merged pull request #14634: URL: https://github.com/apache/airflow/pull/14634 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

[GitHub] [airflow] arkadiuszbach commented on issue #10860: Timeouts in Airflow scheduler logs on AKS

2021-03-05 Thread GitBox
arkadiuszbach commented on issue #10860: URL: https://github.com/apache/airflow/issues/10860#issuecomment-791796674 On Azure LoadBalancers have tcp idle timeout set as default to 4 minutes(it is visible in JSON View), so if you connect and don't interact with it in more than 4 minutes it

[GitHub] [airflow] github-actions[bot] commented on pull request #14634: Fix broken link in LOCAL_VIRTUALENV.rst

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14634: URL: https://github.com/apache/airflow/pull/14634#issuecomment-791796486 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

[GitHub] [airflow] kaxil opened a new pull request #14636: Replace Python 2.7 with 3.6 in IMAGES.rst

2021-03-05 Thread GitBox
kaxil opened a new pull request #14636: URL: https://github.com/apache/airflow/pull/14636 `apache/airflow:v2-0-test-python2.7-ci` does not exists and we do not support Python 2.7 from Airflow 3.6+ --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] github-actions[bot] commented on pull request #14632: Remove unused usage of logging module

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14632: URL: https://github.com/apache/airflow/pull/14632#issuecomment-791792099 [The Workflow run](https://github.com/apache/airflow/actions/runs/625997638) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] zachliu opened a new issue #14635: UpdateMask is useless for POST/PATCH if readOnly=true in the openapi specs

2021-03-05 Thread GitBox
zachliu opened a new issue #14635: URL: https://github.com/apache/airflow/issues/14635 **Apache Airflow version**: 2.0.1 **Environment**: Docker on Linux Mint 20.1, image based on apache/airflow:2.0.1-python3.8 **What happened**: when i made this PATCH request to the API:

[GitHub] [airflow] kaxil opened a new pull request #14634: Fix broken link in LOCAL_VIRTUALENV.rst

2021-03-05 Thread GitBox
kaxil opened a new pull request #14634: URL: https://github.com/apache/airflow/pull/14634 The commits fixes broken link --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] xinbinhuang commented on issue #14511: Test status of provider RCs released 27.02.2021 (RCs)

2021-03-05 Thread GitBox
xinbinhuang commented on issue #14511: URL: https://github.com/apache/airflow/issues/14511#issuecomment-791789786 #14418 is merged, so I think Druid is good to go now (after including the fix) This is an automated message

[GitHub] [airflow] eladkal commented on pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
eladkal commented on pull request #14631: URL: https://github.com/apache/airflow/pull/14631#issuecomment-791789281 We should not suggest to save secrets in extra field if it's not encrypted. We have the same issue with Salesforce https://github.com/apache/airflow/issues/8766

[GitHub] [airflow] mik-laj commented on issue #14597: Provide jinja template syntax to access connections

2021-03-05 Thread GitBox
mik-laj commented on issue #14597: URL: https://github.com/apache/airflow/issues/14597#issuecomment-791788892 > Nothing against that, but it's already very easy for any DAG author to expose any connections secret by simply putting in the DAG It is not good practice for top-level

[GitHub] [airflow] github-actions[bot] commented on pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14492: URL: https://github.com/apache/airflow/pull/14492#issuecomment-791787288 [The Workflow run](https://github.com/apache/airflow/actions/runs/625939193) is cancelling this PR. Building image for the PR has been cancelled

[GitHub] [airflow] github-actions[bot] commented on pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14492: URL: https://github.com/apache/airflow/pull/14492#issuecomment-791787027 [The Workflow run](https://github.com/apache/airflow/actions/runs/625973076) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] kaxil opened a new pull request #14633: Fix broken static check

2021-03-05 Thread GitBox
kaxil opened a new pull request #14633: URL: https://github.com/apache/airflow/pull/14633 Looks like https://github.com/apache/airflow/pull/14627 broke the Master. All tests did not run on that PR (We did not account for the file change in selective checks). Hhence upgrading

[GitHub] [airflow] mik-laj commented on issue #14622: REST API: dag status stream.

2021-03-05 Thread GitBox
mik-laj commented on issue #14622: URL: https://github.com/apache/airflow/issues/14622#issuecomment-791786413 Airflow uses a database for communication, which means there is no efficient way to send messages from the scheduler to the webserver. If we wanted to do it right, we would have

[GitHub] [airflow] ephraimbuddy commented on pull request #14254: Features/sftp to wasb: Transfer to scan files from sftp source path and upload them to Azure Blob Storage

2021-03-05 Thread GitBox
ephraimbuddy commented on pull request #14254: URL: https://github.com/apache/airflow/pull/14254#issuecomment-791786086 Can you rebase again? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] mik-laj edited a comment on issue #14620: Using env in BashOperator results in invalid SyntaxError

2021-03-05 Thread GitBox
mik-laj edited a comment on issue #14620: URL: https://github.com/apache/airflow/issues/14620#issuecomment-791784644 This doesn't look like a problem with Aiirflow, but your script is not compatible with Python 2 and the default system interpreter is Python 2 on your system. You probably

[GitHub] [airflow] mik-laj commented on issue #14620: Using env in BashOperator results in invalid SyntaxError

2021-03-05 Thread GitBox
mik-laj commented on issue #14620: URL: https://github.com/apache/airflow/issues/14620#issuecomment-791784644 This doesn't look like a problem with Aiirflow, but your script is not compatible with Python 2 and the default system interpreter is Python 2 on your system. You probably use

[GitHub] [airflow] kenfinster commented on pull request #14588: Fill DagBag from dag_folder setting for upgrade rules

2021-03-05 Thread GitBox
kenfinster commented on pull request #14588: URL: https://github.com/apache/airflow/pull/14588#issuecomment-791784489 @ephraimbuddy done. is there anything else needed? This is an automated message from the Apache Git

[GitHub] [airflow] mik-laj closed issue #14616: Install airflow with docker No module named 'airflow'

2021-03-05 Thread GitBox
mik-laj closed issue #14616: URL: https://github.com/apache/airflow/issues/14616 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

[GitHub] [airflow] mik-laj commented on issue #14616: Install airflow with docker No module named 'airflow'

2021-03-05 Thread GitBox
mik-laj commented on issue #14616: URL: https://github.com/apache/airflow/issues/14616#issuecomment-791782926 Duplicate: #14520 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] github-actions[bot] commented on pull request #14632: Remove unused usage of logging module

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14632: URL: https://github.com/apache/airflow/pull/14632#issuecomment-791782824 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] github-actions[bot] commented on pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14631: URL: https://github.com/apache/airflow/pull/14631#issuecomment-791782163 [The Workflow run](https://github.com/apache/airflow/actions/runs/625944985) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] kaxil opened a new pull request #14632: Remove unused usage of logging module

2021-03-05 Thread GitBox
kaxil opened a new pull request #14632: URL: https://github.com/apache/airflow/pull/14632 Unless I am missing something the logging module was imported in the following files but was not used: - airflow/api_connexion/endpoints/dag_source_endpoint.py -

[GitHub] [airflow] github-actions[bot] commented on pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14631: URL: https://github.com/apache/airflow/pull/14631#issuecomment-791781322 [The Workflow run](https://github.com/apache/airflow/actions/runs/625903367) is cancelling this PR. Building image for the PR has been cancelled

[GitHub] [airflow] github-actions[bot] commented on pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14631: URL: https://github.com/apache/airflow/pull/14631#issuecomment-791781061 [The Workflow run](https://github.com/apache/airflow/actions/runs/625939193) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] JavierLopezT closed pull request #13152: Convert columns in get_pandas_df to lowercase SnowflakeHook

2021-03-05 Thread GitBox
JavierLopezT closed pull request #13152: URL: https://github.com/apache/airflow/pull/13152 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

[GitHub] [airflow] JavierLopezT commented on pull request #13152: Convert columns in get_pandas_df to lowercase SnowflakeHook

2021-03-05 Thread GitBox
JavierLopezT commented on pull request #13152: URL: https://github.com/apache/airflow/pull/13152#issuecomment-791772817 > Yeah. The new version of python connector is out and we switched to it. But after looking closely - I also think we should not merge this one - similarly to Daniel,

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
boring-cyborg[bot] commented on pull request #14631: URL: https://github.com/apache/airflow/pull/14631#issuecomment-791770889 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

[GitHub] [airflow] sunkickr opened a new pull request #14631: Add Dynamic fields to Snowflake Connection

2021-03-05 Thread GitBox
sunkickr opened a new pull request #14631: URL: https://github.com/apache/airflow/pull/14631 Adds form fields and custom form behavior for the Snowflake connection so that it is more obvious to new users what fields need to be filled out. I have divided the inputs into three main

[airflow] branch master updated (5906540 -> 4589375)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 5906540 Bugfix: DruidOperator fails to submit ingestion tasks (#14418) add 4589375 Fix various links in

[airflow] branch master updated (c2a0cb9 -> 5906540)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from c2a0cb9 Rename last_scheduler_run into last_parsed_time, and ensure it's updated in DB (#14581) add

[GitHub] [airflow] github-actions[bot] commented on pull request #14630: Fix various links in README.md

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14630: URL: https://github.com/apache/airflow/pull/14630#issuecomment-791770395 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

[GitHub] [airflow] kaxil merged pull request #14630: Fix various links in README.md

2021-03-05 Thread GitBox
kaxil merged pull request #14630: URL: https://github.com/apache/airflow/pull/14630 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

[GitHub] [airflow] kaxil commented on pull request #14418: Fix #14417 : Failing to submit Druid ingestion task

2021-03-05 Thread GitBox
kaxil commented on pull request #14418: URL: https://github.com/apache/airflow/pull/14418#issuecomment-791770160 Thanks @TheRum and @xinbinhuang This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] kaxil merged pull request #14418: Fix #14417 : Failing to submit Druid ingestion task

2021-03-05 Thread GitBox
kaxil merged pull request #14418: URL: https://github.com/apache/airflow/pull/14418 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

[GitHub] [airflow] kaxil closed issue #14417: DruidOperator failing to submit ingestion tasks : Getting 500 error code from Druid

2021-03-05 Thread GitBox
kaxil closed issue #14417: URL: https://github.com/apache/airflow/issues/14417 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

[GitHub] [airflow] github-actions[bot] commented on pull request #14418: Fix #14417 : Failing to submit Druid ingestion task

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14418: URL: https://github.com/apache/airflow/pull/14418#issuecomment-791769989 The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not

[GitHub] [airflow] ashb commented on a change in pull request #14380: Prevent mixed case env vars from crashing a worker

2021-03-05 Thread GitBox
ashb commented on a change in pull request #14380: URL: https://github.com/apache/airflow/pull/14380#discussion_r588768950 ## File path: tests/core/test_configuration.py ## @@ -80,6 +80,15 @@ def test_case_sensitivity(self): assert conf.get("core", "PERCENT") ==

[GitHub] [airflow] github-actions[bot] commented on pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14492: URL: https://github.com/apache/airflow/pull/14492#issuecomment-791766204 [The Workflow run](https://github.com/apache/airflow/actions/runs/625797986) is cancelling this PR. Building image for the PR has been cancelled

[GitHub] [airflow] github-actions[bot] commented on pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14492: URL: https://github.com/apache/airflow/pull/14492#issuecomment-791765979 [The Workflow run](https://github.com/apache/airflow/actions/runs/625877636) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static

[GitHub] [airflow] kaxil opened a new pull request #14630: Fix various links in README.md

2021-03-05 Thread GitBox
kaxil opened a new pull request #14630: URL: https://github.com/apache/airflow/pull/14630 - Committer's guide was broken - Some links pointed to old url - Added reference to scheduler docs --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] haard commented on a change in pull request #14380: Prevent mixed case env vars from crashing a worker

2021-03-05 Thread GitBox
haard commented on a change in pull request #14380: URL: https://github.com/apache/airflow/pull/14380#discussion_r588754638 ## File path: tests/core/test_configuration.py ## @@ -80,6 +80,15 @@ def test_case_sensitivity(self): assert conf.get("core", "PERCENT") ==

[GitHub] [airflow] XD-DENG commented on pull request #14581: Rename last_scheduler_run into last_parsed_time, and ensure it's updated in DB

2021-03-05 Thread GitBox
XD-DENG commented on pull request #14581: URL: https://github.com/apache/airflow/pull/14581#issuecomment-791746481 Regarding the milestone: Given this is mainly (at least initially) intended to fix and ensure the time to be updated in DB properly, I still consider it as a bugfix so I

[GitHub] [airflow] dstandish commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
dstandish commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r588750313 ## File path: airflow/providers/airbyte/hooks/airbyte.py ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] marcosmarxm commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
marcosmarxm commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r588746034 ## File path: airflow/providers/airbyte/hooks/airbyte.py ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

[airflow] branch master updated (d8bd107 -> c2a0cb9)

2021-03-05 Thread xddeng
This is an automated email from the ASF dual-hosted git repository. xddeng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from d8bd107 Update pre-commit hooks (#14627) add c2a0cb9 Rename last_scheduler_run into last_parsed_time, and

[GitHub] [airflow] XD-DENG merged pull request #14581: Rename last_scheduler_run into last_parsed_time, and ensure it's updated in DB

2021-03-05 Thread GitBox
XD-DENG merged pull request #14581: URL: https://github.com/apache/airflow/pull/14581 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

[GitHub] [airflow] XD-DENG commented on pull request #14581: Rename last_scheduler_run into last_parsed_time, and ensure it's updated in DB

2021-03-05 Thread GitBox
XD-DENG commented on pull request #14581: URL: https://github.com/apache/airflow/pull/14581#issuecomment-791740023 Finally, CI is green! This is an automated message from the Apache Git Service. To respond to the message,

[jira] [Commented] (AIRFLOW-5655) Incorrect capitalization of env var causes task start to fail

2021-03-05 Thread Jorge Machado (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17296355#comment-17296355 ] Jorge Machado commented on AIRFLOW-5655: I can confirm that I had the same error. Fixed by

[GitHub] [airflow] ephraimbuddy commented on pull request #14588: Fill DagBag from dag_folder setting for upgrade rules

2021-03-05 Thread GitBox
ephraimbuddy commented on pull request #14588: URL: https://github.com/apache/airflow/pull/14588#issuecomment-791730383 > Hi @kenfinster , most likely the error has nothing to do with your changes, because the same errors already exist in the latest commit in branch `v1-10-stable`

[airflow] branch master updated (92e2dd8 -> d8bd107)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 92e2dd8 Update filepaths in boring-cyborg.yml (#14628) add d8bd107 Update pre-commit hooks (#14627) No

[airflow] branch master updated (33910d6 -> 92e2dd8)

2021-03-05 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 33910d6 Default to Celery Task model when backend model does not exist (#14612) add 92e2dd8 Update

[GitHub] [airflow] kaxil merged pull request #14628: Update filepaths in boring-cyborg.yml

2021-03-05 Thread GitBox
kaxil merged pull request #14628: URL: https://github.com/apache/airflow/pull/14628 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

[GitHub] [airflow] kaxil merged pull request #14627: Update pre-commit hooks

2021-03-05 Thread GitBox
kaxil merged pull request #14627: URL: https://github.com/apache/airflow/pull/14627 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

[GitHub] [airflow] dstandish commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
dstandish commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r588726043 ## File path: airflow/providers/airbyte/hooks/airbyte.py ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] dstandish commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
dstandish commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r588723220 ## File path: airflow/providers/airbyte/hooks/airbyte.py ## @@ -0,0 +1,100 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or

[GitHub] [airflow] marcosmarxm commented on a change in pull request #14492: Adds new community provider Airbyte

2021-03-05 Thread GitBox
marcosmarxm commented on a change in pull request #14492: URL: https://github.com/apache/airflow/pull/14492#discussion_r588720598 ## File path: airflow/providers/airbyte/hooks/airbyte.py ## @@ -0,0 +1,100 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] [airflow] ashb commented on pull request #14629: Improve readability of Sensor Schedule Mode table

2021-03-05 Thread GitBox
ashb commented on pull request #14629: URL: https://github.com/apache/airflow/pull/14629#issuecomment-791717865 Could you provide before and after screenshots please? Would help to review this This is an automated message

[GitHub] [airflow] github-actions[bot] commented on pull request #14628: Update filepaths in boring-cyborg.yml

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14628: URL: https://github.com/apache/airflow/pull/14628#issuecomment-791717759 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

[GitHub] [airflow] github-actions[bot] commented on pull request #14627: Update pre-commit hooks

2021-03-05 Thread GitBox
github-actions[bot] commented on pull request #14627: URL: https://github.com/apache/airflow/pull/14627#issuecomment-791717522 The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide

  1   2   3   >