[jira] [Created] (AIRFLOW-3432) Test is missing for [AIRFLOW-3233](PR 4069)

2018-12-01 Thread Xiaodong DENG (JIRA)
Xiaodong DENG created AIRFLOW-3432: -- Summary: Test is missing for [AIRFLOW-3233](PR 4069) Key: AIRFLOW-3432 URL: https://issues.apache.org/jira/browse/AIRFLOW-3432 Project: Apache Airflow

[jira] [Commented] (AIRFLOW-3426) Correct references to Python version tested (3.4 -> 3.5)

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706111#comment-16706111 ] ASF GitHub Bot commented on AIRFLOW-3426: - feng-tao closed pull request #4259: [AIRFLOW-3426]

[GitHub] feng-tao closed pull request #4259: [AIRFLOW-3426] Bugfix / Correct Python Version Documentation Reference

2018-12-01 Thread GitBox
feng-tao closed pull request #4259: [AIRFLOW-3426] Bugfix / Correct Python Version Documentation Reference URL: https://github.com/apache/incubator-airflow/pull/4259 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] feng-tao commented on issue #4259: [AIRFLOW-3426] Bugfix / Correct Python Version Documentation Reference

2018-12-01 Thread GitBox
feng-tao commented on issue #4259: [AIRFLOW-3426] Bugfix / Correct Python Version Documentation Reference URL: https://github.com/apache/incubator-airflow/pull/4259#issuecomment-443484030 lgtm This is an automated message

[GitHub] feng-tao commented on issue #4262: [AIRFLOW-3431] Document how to report security vulnerabilities.

2018-12-01 Thread GitBox
feng-tao commented on issue #4262: [AIRFLOW-3431] Document how to report security vulnerabilities. URL: https://github.com/apache/incubator-airflow/pull/4262#issuecomment-443483019 This file is more like doc for airflow security feature. Should we put this into contribute file?

[GitHub] caddac commented on issue #3684: [AIRFLOW-2840] - add update connections cli option

2018-12-01 Thread GitBox
caddac commented on issue #3684: [AIRFLOW-2840] - add update connections cli option URL: https://github.com/apache/incubator-airflow/pull/3684#issuecomment-443481367 @ron819 I'm going to address this bug in my PR. @ashb I'm still looking for some clarity on what is going on with

[jira] [Commented] (AIRFLOW-3406) Implement an Azure CosmosDB operator

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706088#comment-16706088 ] ASF GitHub Bot commented on AIRFLOW-3406: - tmiller-msft opened a new pull request #4265:

[GitHub] tmiller-msft opened a new pull request #4265: [AIRFLOW-3406] Implement an Azure CosmosDB operator

2018-12-01 Thread GitBox
tmiller-msft opened a new pull request #4265: [AIRFLOW-3406] Implement an Azure CosmosDB operator URL: https://github.com/apache/incubator-airflow/pull/4265 Add an operator and hook to manipulate and use Azure CosmosDB documents, including creation, deletion, and updating documents

[jira] [Commented] (AIRFLOW-3406) Implement an Azure CosmosDB operator

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706087#comment-16706087 ] ASF GitHub Bot commented on AIRFLOW-3406: - tmiller-msft closed pull request #4264:

[GitHub] tmiller-msft closed pull request #4264: [AIRFLOW-3406] Implement an Azure CosmosDB operator

2018-12-01 Thread GitBox
tmiller-msft closed pull request #4264: [AIRFLOW-3406] Implement an Azure CosmosDB operator URL: https://github.com/apache/incubator-airflow/pull/4264 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] caddac commented on a change in pull request #3684: [AIRFLOW-2840] - add update connections cli option

2018-12-01 Thread GitBox
caddac commented on a change in pull request #3684: [AIRFLOW-2840] - add update connections cli option URL: https://github.com/apache/incubator-airflow/pull/3684#discussion_r238084129 ## File path: airflow/api/client/local_client.py ## @@ -51,3 +51,83 @@ def

[jira] [Commented] (AIRFLOW-3406) Implement an Azure CosmosDB operator

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706073#comment-16706073 ] ASF GitHub Bot commented on AIRFLOW-3406: - tmiller-msft opened a new pull request #4264:

[GitHub] tmiller-msft opened a new pull request #4264: [AIRFLOW-3406] Implement an Azure CosmosDB operator

2018-12-01 Thread GitBox
tmiller-msft opened a new pull request #4264: [AIRFLOW-3406] Implement an Azure CosmosDB operator URL: https://github.com/apache/incubator-airflow/pull/4264 Add an operator and hook to manipulate and use Azure CosmosDB documents, including creation, deletion, and updating documents

[GitHub] jmcarp commented on a change in pull request #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

2018-12-01 Thread GitBox
jmcarp commented on a change in pull request #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#discussion_r238083062 ## File path: tests/executors/test_celery_executor.py ## @@ -38,48 +40,80 @@

[GitHub] KevinYang21 commented on issue #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent

2018-12-01 Thread GitBox
KevinYang21 commented on issue #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent URL: https://github.com/apache/incubator-airflow/pull/4253#issuecomment-443471365 @kaxil Thank you for reviewing! Replied the comments in their own thread.

[GitHub] KevinYang21 commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent

2018-12-01 Thread GitBox
KevinYang21 commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent URL: https://github.com/apache/incubator-airflow/pull/4253#discussion_r238081291 ## File path: airflow/settings.py ## @@ -261,7 +261,7 @@ def

[GitHub] KevinYang21 commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent

2018-12-01 Thread GitBox
KevinYang21 commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent URL: https://github.com/apache/incubator-airflow/pull/4253#discussion_r238081263 ## File path: airflow/utils/dag_processing.py ## @@ -539,7 +541,9 @@ def

[GitHub] ashb opened a new pull request #4263: [AIRFLOW-XXX] Fix display of SageMaker operators/hook docs

2018-12-01 Thread GitBox
ashb opened a new pull request #4263: [AIRFLOW-XXX] Fix display of SageMaker operators/hook docs URL: https://github.com/apache/incubator-airflow/pull/4263 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-3431) Document how to report security vulnerabilities and issues safely

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706032#comment-16706032 ] ASF GitHub Bot commented on AIRFLOW-3431: - ashb opened a new pull request #4262: [AIRFLOW-3431]

[GitHub] ashb opened a new pull request #4262: [AIRFLOW-3431] Document how to report security vulnerabilities.

2018-12-01 Thread GitBox
ashb opened a new pull request #4262: [AIRFLOW-3431] Document how to report security vulnerabilities. URL: https://github.com/apache/incubator-airflow/pull/4262 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244#issuecomment-442078711 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4244?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244#issuecomment-442078711 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4244?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
codecov-io edited a comment on issue #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244#issuecomment-442078711 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4244?src=pr=h1) Report > Merging

[jira] [Commented] (AIRFLOW-3396) Redundant sql param check in BigQueryOperator

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706013#comment-16706013 ] ASF GitHub Bot commented on AIRFLOW-3396: - kaxil closed pull request #4224: [AIRFLOW-3396] Make

[jira] [Resolved] (AIRFLOW-3396) Redundant sql param check in BigQueryOperator

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-3396. - Resolution: Fixed Resolved by https://github.com/apache/incubator-airflow/pull/4224 >

[GitHub] kaxil closed pull request #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator

2018-12-01 Thread GitBox
kaxil closed pull request #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator URL: https://github.com/apache/incubator-airflow/pull/4224 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[jira] [Resolved] (AIRFLOW-3403) Add AWS Athena Sensor

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3403?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-3403. - Resolution: Fixed Fix Version/s: 1.10.2 Resolved by

[jira] [Updated] (AIRFLOW-3403) Add AWS Athena Sensor

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3403?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik updated AIRFLOW-3403: Summary: Add AWS Athena Sensor (was: create athena sensor) > Add AWS Athena Sensor >

[jira] [Updated] (AIRFLOW-3403) Add AWS Athena Sensor

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3403?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik updated AIRFLOW-3403: Issue Type: New Feature (was: Improvement) > Add AWS Athena Sensor > - > >

[jira] [Commented] (AIRFLOW-3403) create athena sensor

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706006#comment-16706006 ] ASF GitHub Bot commented on AIRFLOW-3403: - kaxil closed pull request #4244: [AIRFLOW-3403]

[GitHub] kaxil commented on issue #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
kaxil commented on issue #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244#issuecomment-443459050 Thanks, @tal181 for your contribution :) Thanks @XD-DENG @feluelle for help in reviewing

[GitHub] kaxil closed pull request #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
kaxil closed pull request #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] kaxil commented on a change in pull request #4244: [AIRFLOW-3403] Create Athena sensor

2018-12-01 Thread GitBox
kaxil commented on a change in pull request #4244: [AIRFLOW-3403] Create Athena sensor URL: https://github.com/apache/incubator-airflow/pull/4244#discussion_r238076583 ## File path: airflow/contrib/sensors/aws_athena_sensor.py ## @@ -0,0 +1,79 @@ +# -*- coding: utf-8 -*-

[GitHub] kaxil edited a comment on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator

2018-12-01 Thread GitBox
kaxil edited a comment on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator URL: https://github.com/apache/incubator-airflow/pull/4224#issuecomment-443457478 @ashb We issued deprecation warning in 1.10.0:

[GitHub] kaxil edited a comment on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator

2018-12-01 Thread GitBox
kaxil edited a comment on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator URL: https://github.com/apache/incubator-airflow/pull/4224#issuecomment-443457478 @ashb We issued deprecation warning in 1.10.0:

[GitHub] kaxil commented on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator

2018-12-01 Thread GitBox
kaxil commented on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator URL: https://github.com/apache/incubator-airflow/pull/4224#issuecomment-443457478 @ashb We deprecated it in 1.10.0:

[GitHub] kaxil commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent

2018-12-01 Thread GitBox
kaxil commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent URL: https://github.com/apache/incubator-airflow/pull/4253#discussion_r238076020 ## File path: airflow/utils/dag_processing.py ## @@ -539,7 +541,9 @@ def helper():

[GitHub] kaxil commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent

2018-12-01 Thread GitBox
kaxil commented on a change in pull request #4253: [AIRFLOW-3414] Fix reload_module in DagFileProcessorAgent URL: https://github.com/apache/incubator-airflow/pull/4253#discussion_r238076043 ## File path: airflow/settings.py ## @@ -261,7 +261,7 @@ def

[GitHub] kaxil commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
kaxil commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261#issuecomment-443456217 yes that is true @sokoloff  Thanks This is

[jira] [Assigned] (AIRFLOW-3431) Document how to report security vulnerabilities and issues safely

2018-12-01 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3431?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor reassigned AIRFLOW-3431: -- Assignee: Ash Berlin-Taylor > Document how to report security

[jira] [Assigned] (AIRFLOW-3430) Document how to become a commiter

2018-12-01 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3430?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor reassigned AIRFLOW-3430: -- Assignee: Fokko Driesprong (was: Ash Berlin-Taylor) > Document how to

[jira] [Created] (AIRFLOW-3431) Document how to report security vulnerabilities and issues safely

2018-12-01 Thread Ash Berlin-Taylor (JIRA)
Ash Berlin-Taylor created AIRFLOW-3431: -- Summary: Document how to report security vulnerabilities and issues safely Key: AIRFLOW-3431 URL: https://issues.apache.org/jira/browse/AIRFLOW-3431

[jira] [Created] (AIRFLOW-3430) Document how to become a commiter

2018-12-01 Thread Ash Berlin-Taylor (JIRA)
Ash Berlin-Taylor created AIRFLOW-3430: -- Summary: Document how to become a commiter Key: AIRFLOW-3430 URL: https://issues.apache.org/jira/browse/AIRFLOW-3430 Project: Apache Airflow

[GitHub] ashb commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters

2018-12-01 Thread GitBox
ashb commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters URL: https://github.com/apache/incubator-airflow/pull/4201#discussion_r238074516 ## File path: airflow/www_rbac/templates/airflow/dags.html ## @@

[GitHub] ashb commented on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator

2018-12-01 Thread GitBox
ashb commented on issue #4224: [AIRFLOW-3396] Make sql param as required in BigQueryOperator URL: https://github.com/apache/incubator-airflow/pull/4224#issuecomment-443452526 When did we release the change that deprecated/removed BQL? The code change is fine, just wondering about

[GitHub] ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api.

2018-12-01 Thread GitBox
ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api. URL: https://github.com/apache/incubator-airflow/pull/4232#discussion_r238073949 ## File path: airflow/api/common/experimental/connection.py ## @@ -0,0 +1,77 @@ +# -*- coding: utf-8

[GitHub] ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api.

2018-12-01 Thread GitBox
ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api. URL: https://github.com/apache/incubator-airflow/pull/4232#discussion_r238074045 ## File path: airflow/api/common/experimental/connection.py ## @@ -0,0 +1,77 @@ +# -*- coding: utf-8

[GitHub] ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api.

2018-12-01 Thread GitBox
ashb commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api. URL: https://github.com/apache/incubator-airflow/pull/4232#discussion_r238074020 ## File path: airflow/api/common/experimental/connection.py ## @@ -0,0 +1,77 @@ +# -*- coding: utf-8

[GitHub] ashb commented on issue #4163: [AIRFLOW-3319] - KubernetsExecutor: Need in try_number in labels if getting them later

2018-12-01 Thread GitBox
ashb commented on issue #4163: [AIRFLOW-3319] - KubernetsExecutor: Need in try_number in labels if getting them later URL: https://github.com/apache/incubator-airflow/pull/4163#issuecomment-443451150 @verdan Does this fix look right to you? I know you created a branch

[GitHub] ashb commented on a change in pull request #4207: [AIRFLOW-3367] Run celery integration test with redis broker.

2018-12-01 Thread GitBox
ashb commented on a change in pull request #4207: [AIRFLOW-3367] Run celery integration test with redis broker. URL: https://github.com/apache/incubator-airflow/pull/4207#discussion_r238073600 ## File path: tests/executors/test_celery_executor.py ## @@ -38,48 +40,80 @@

[GitHub] sokoloff commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
sokoloff commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261#issuecomment-443450287 @kaxil Yes, it checked for `is not None` but not for empty string.

[GitHub] codecov-io edited a comment on issue #4260: [AIRFLOW-XXX] Add missing GCP operators to Docs

2018-12-01 Thread GitBox
codecov-io edited a comment on issue #4260: [AIRFLOW-XXX] Add missing GCP operators to Docs URL: https://github.com/apache/incubator-airflow/pull/4260#issuecomment-443450083 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4260?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4260: [AIRFLOW-XXX] Add missing GCP operators to Docs

2018-12-01 Thread GitBox
codecov-io commented on issue #4260: [AIRFLOW-XXX] Add missing GCP operators to Docs URL: https://github.com/apache/incubator-airflow/pull/4260#issuecomment-443450083 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4260?src=pr=h1) Report > Merging

[jira] [Updated] (AIRFLOW-3323) Support Basic Authentication for Flower

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik updated AIRFLOW-3323: Fix Version/s: (was: 2.0.0) 1.10.2 > Support Basic Authentication for

[GitHub] kaxil closed pull request #4034: Update UPDATING.md

2018-12-01 Thread GitBox
kaxil closed pull request #4034: Update UPDATING.md URL: https://github.com/apache/incubator-airflow/pull/4034 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[jira] [Commented] (AIRFLOW-3425) GoogleCloudBaseHook doesn't set default scope when google_cloud_default connection is empty

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16705951#comment-16705951 ] Kaxil Naik commented on AIRFLOW-3425: - Btw [~alsokoloff] it is not a bug, it already checked for

[GitHub] kaxil commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
kaxil commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261#issuecomment-443447697 Btw @sokoloff it is not a bug, it already checked for `is not None`, and not `is None` as you mentioned.

[jira] [Commented] (AIRFLOW-3425) GoogleCloudBaseHook doesn't set default scope when google_cloud_default connection is empty

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16705947#comment-16705947 ] ASF GitHub Bot commented on AIRFLOW-3425: - kaxil closed pull request #4261: [AIRFLOW-3425] Fix

[jira] [Resolved] (AIRFLOW-3425) GoogleCloudBaseHook doesn't set default scope when google_cloud_default connection is empty

2018-12-01 Thread Kaxil Naik (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3425?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-3425. - Resolution: Fixed Fix Version/s: 2.0.0 Resolved by

[GitHub] kaxil closed pull request #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
kaxil closed pull request #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[jira] [Commented] (AIRFLOW-3036) Upgrading to Airflow 1.10 not possible using GCP Cloud SQL for MYSQL

2018-12-01 Thread Roger Kaufmann (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16705871#comment-16705871 ] Roger Kaufmann commented on AIRFLOW-3036: - I ran into this issue too on Swisscom Cloud Foundry

[GitHub] codecov-io commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
codecov-io commented on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261#issuecomment-443433996 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4261?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
codecov-io edited a comment on issue #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261#issuecomment-443433996 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4261?src=pr=h1) Report >

[GitHub] XD-DENG commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api.

2018-12-01 Thread GitBox
XD-DENG commented on a change in pull request #4232: [AIRFLOW-3390] Add connections to api. URL: https://github.com/apache/incubator-airflow/pull/4232#discussion_r238065671 ## File path: airflow/api/common/experimental/connection.py ## @@ -0,0 +1,77 @@ +# -*- coding:

[jira] [Commented] (AIRFLOW-3425) GoogleCloudBaseHook doesn't set default scope when google_cloud_default connection is empty

2018-12-01 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16705857#comment-16705857 ] ASF GitHub Bot commented on AIRFLOW-3425: - sokoloff opened a new pull request #4261:

[GitHub] sokoloff opened a new pull request #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook

2018-12-01 Thread GitBox
sokoloff opened a new pull request #4261: [AIRFLOW-3425] Fix default scope setting in GoogleCloudBaseHook URL: https://github.com/apache/incubator-airflow/pull/4261 ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW) issues and

[GitHub] mik-laj commented on issue #3000: [AIRFLOW-2056] Refactor: Integrate all GCS operators into 1 file

2018-12-01 Thread GitBox
mik-laj commented on issue #3000: [AIRFLOW-2056] Refactor: Integrate all GCS operators into 1 file URL: https://github.com/apache/incubator-airflow/pull/3000#issuecomment-443422430 @kaxil @criccomini I propose to introduce a transitional period in which operators will be available from

[GitHub] mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters

2018-12-01 Thread GitBox
mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters URL: https://github.com/apache/incubator-airflow/pull/4201#discussion_r238061495 ## File path: airflow/www_rbac/templates/airflow/dags.html ## @@

[GitHub] mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters

2018-12-01 Thread GitBox
mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters URL: https://github.com/apache/incubator-airflow/pull/4201#discussion_r238061428 ## File path: airflow/www/templates/airflow/dags.html ## @@

[GitHub] mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters

2018-12-01 Thread GitBox
mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters URL: https://github.com/apache/incubator-airflow/pull/4201#discussion_r238061378 ## File path: airflow/www/templates/airflow/dags.html ## @@

[GitHub] mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters

2018-12-01 Thread GitBox
mik-laj commented on a change in pull request #4201: [AIRFLOW-3360] Make the DAGs search respect other querystring parameters URL: https://github.com/apache/incubator-airflow/pull/4201#discussion_r238061268 ## File path: airflow/www/templates/airflow/dags.html ## @@

[GitHub] reubenvanammers commented on a change in pull request #4156: [AIRFLOW-3314] Changed auto inlets feature to work as described

2018-12-01 Thread GitBox
reubenvanammers commented on a change in pull request #4156: [AIRFLOW-3314] Changed auto inlets feature to work as described URL: https://github.com/apache/incubator-airflow/pull/4156#discussion_r238060986 ## File path: airflow/lineage/__init__.py ## @@ -110,26 +114,31 @@

[GitHub] matrixise commented on issue #3723: [AIRFLOW-2876] Update Tenacity to 4.12

2018-12-01 Thread GitBox
matrixise commented on issue #3723: [AIRFLOW-2876] Update Tenacity to 4.12 URL: https://github.com/apache/incubator-airflow/pull/3723#issuecomment-443416300 +1 for the support of tenacity and python 3.7. 3.7 has been released in june, and we are in november. Python 3.7 is the default

[GitHub] oelesinsc24 commented on issue #4068: [AIRFLOW-2310]: Add AWS Glue Job Compatibility to Airflow

2018-12-01 Thread GitBox
oelesinsc24 commented on issue #4068: [AIRFLOW-2310]: Add AWS Glue Job Compatibility to Airflow URL: https://github.com/apache/incubator-airflow/pull/4068#issuecomment-443410986 > Is there any further update on when this PR will be actioned on closed ? As far as I know, the PR is