[GitHub] sbilinski opened a new pull request #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs

2018-09-18 Thread GitBox
sbilinski opened a new pull request #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs URL: https://github.com/apache/incubator-airflow/pull/3915 ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW/) issues and references them i

[GitHub] codecov-io commented on issue #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs

2018-09-18 Thread GitBox
codecov-io commented on issue #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs URL: https://github.com/apache/incubator-airflow/pull/3915#issuecomment-422586211 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3915?src=pr&el=h1) Report > Merging [#3915](https://codeco

[GitHub] codecov-io commented on issue #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs

2018-09-18 Thread GitBox
codecov-io commented on issue #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs URL: https://github.com/apache/incubator-airflow/pull/3915#issuecomment-422586261 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3915?src=pr&el=h1) Report > Merging [#3915](https://codeco

[GitHub] jgao54 opened a new pull request #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-18 Thread GitBox
jgao54 opened a new pull request #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](https://issues.a

[GitHub] codecov-io commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-18 Thread GitBox
codecov-io commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422598348 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3916?src=pr&el=h1) Report > Merging [#3916](ht

[GitHub] codecov-io commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-18 Thread GitBox
codecov-io commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422598371 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3916?src=pr&el=h1) Report > Merging [#3916](ht

[GitHub] jmcarp opened a new pull request #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py.

2018-09-18 Thread GitBox
jmcarp opened a new pull request #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py. URL: https://github.com/apache/incubator-airflow/pull/3917 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://is

[GitHub] codecov-io commented on issue #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py.

2018-09-18 Thread GitBox
codecov-io commented on issue #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py. URL: https://github.com/apache/incubator-airflow/pull/3917#issuecomment-422625841 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3917?src=pr&el=h1) Report > Merging [#39

[GitHub] codecov-io commented on issue #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py.

2018-09-18 Thread GitBox
codecov-io commented on issue #3917: [AIRFLOW-3086] Add extras group for google auth to setup.py. URL: https://github.com/apache/incubator-airflow/pull/3917#issuecomment-422625851 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3917?src=pr&el=h1) Report > Merging [#39

[GitHub] criccomini commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-18 Thread GitBox
criccomini commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422627689 LGTM! This is an automated message from the Apache

[GitHub] mishikaSingh commented on issue #3650: [AIRFLOW-2703] exceptions from scheduler's heartbeat is handled so that scheduler does not crash

2018-09-19 Thread GitBox
mishikaSingh commented on issue #3650: [AIRFLOW-2703] exceptions from scheduler's heartbeat is handled so that scheduler does not crash URL: https://github.com/apache/incubator-airflow/pull/3650#issuecomment-422683361 @feng-tao can you please review the changes? --

[GitHub] kaxil closed pull request #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs

2018-09-19 Thread GitBox
kaxil closed pull request #3915: [AIRFLOW-XXX] Fix SlackWebhookOperator docs URL: https://github.com/apache/incubator-airflow/pull/3915 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

[GitHub] sethwoodworth opened a new pull request #3918: [AIRFLOW-3088] Include slack-compatible emoji image.

2018-09-19 Thread GitBox
sethwoodworth opened a new pull request #3918: [AIRFLOW-3088] Include slack-compatible emoji image. URL: https://github.com/apache/incubator-airflow/pull/3918 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://iss

[GitHub] jmcarp opened a new pull request #3919: [AIRFLOW-3089] Drop hard-coded url scheme in google auth redirect.

2018-09-19 Thread GitBox
jmcarp opened a new pull request #3919: [AIRFLOW-3089] Drop hard-coded url scheme in google auth redirect. URL: https://github.com/apache/incubator-airflow/pull/3919 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](http

[GitHub] TrevorEdwards commented on issue #2466: [AIRFLOW-1441] Fix inconsistent tutorial code

2018-09-19 Thread GitBox
TrevorEdwards commented on issue #2466: [AIRFLOW-1441] Fix inconsistent tutorial code URL: https://github.com/apache/incubator-airflow/pull/2466#issuecomment-422866524 Updated to set the right link. Also added days=1 based on https://github.com/apache/incubator-airflow/blob/master/

[GitHub] codecov-io commented on issue #3919: [AIRFLOW-3089] Drop hard-coded url scheme in google auth redirect.

2018-09-19 Thread GitBox
codecov-io commented on issue #3919: [AIRFLOW-3089] Drop hard-coded url scheme in google auth redirect. URL: https://github.com/apache/incubator-airflow/pull/3919#issuecomment-422879617 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3919?src=pr&el=h1) Report > Merging

[GitHub] jgao54 commented on issue #3913: [AIRFLOW-3072] Assign permission get_logs_with_metadata to viewer role

2018-09-19 Thread GitBox
jgao54 commented on issue #3913: [AIRFLOW-3072] Assign permission get_logs_with_metadata to viewer role URL: https://github.com/apache/incubator-airflow/pull/3913#issuecomment-422884762 imho, viewer was originally intended for anything read-only, which would include log-reading, while use

[GitHub] wrp opened a new pull request #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug

2018-09-19 Thread GitBox
wrp opened a new pull request #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug URL: https://github.com/apache/incubator-airflow/pull/3920 Make sure you have checked _all_ steps below. ### Jira - [ x] My PR addresses the following [Airflow Jira](https://issues

[GitHub] jgao54 commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-19 Thread GitBox
jgao54 commented on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422907996 Closing because it's a dupe of AIRFLOW-3072. This is a

[GitHub] jgao54 closed pull request #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-19 Thread GitBox
jgao54 closed pull request #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916 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:

[GitHub] jgao54 edited a comment on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-19 Thread GitBox
jgao54 edited a comment on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422907996 Closing because it's a dupe of #3916 3072. This

[GitHub] jgao54 edited a comment on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI

2018-09-19 Thread GitBox
jgao54 edited a comment on issue #3916: [AIRFLOW-3085] Bug fix to allow log display in RBAC UI URL: https://github.com/apache/incubator-airflow/pull/3916#issuecomment-422907996 Closing because it's a dupe of #3913 This is

[GitHub] wrp opened a new pull request #3921: [AIRFLOW-3090] Specify path of key file in log message

2018-09-19 Thread GitBox
wrp opened a new pull request #3921: [AIRFLOW-3090] Specify path of key file in log message URL: https://github.com/apache/incubator-airflow/pull/3921 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apac

[GitHub] Fokko closed pull request #2466: [AIRFLOW-1441] Fix inconsistent tutorial code

2018-09-19 Thread GitBox
Fokko closed pull request #2466: [AIRFLOW-1441] Fix inconsistent tutorial code URL: https://github.com/apache/incubator-airflow/pull/2466 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

[GitHub] Fokko commented on issue #2466: [AIRFLOW-1441] Fix inconsistent tutorial code

2018-09-19 Thread GitBox
Fokko commented on issue #2466: [AIRFLOW-1441] Fix inconsistent tutorial code URL: https://github.com/apache/incubator-airflow/pull/2466#issuecomment-422916261 Thanks @TrevorEdwards Looks good! 👍 This is an automated message

[GitHub] codecov-io commented on issue #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug

2018-09-19 Thread GitBox
codecov-io commented on issue #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug URL: https://github.com/apache/incubator-airflow/pull/3920#issuecomment-422921857 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3920?src=pr&el=h1) Report > Merging [#3920

[GitHub] codecov-io edited a comment on issue #3921: [AIRFLOW-3090] Specify path of key file in log message

2018-09-19 Thread GitBox
codecov-io edited a comment on issue #3921: [AIRFLOW-3090] Specify path of key file in log message URL: https://github.com/apache/incubator-airflow/pull/3921#issuecomment-422936030 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3921?src=pr&el=h1) Report > Merging [#3

[GitHub] codecov-io commented on issue #3921: [AIRFLOW-3090] Specify path of key file in log message

2018-09-19 Thread GitBox
codecov-io commented on issue #3921: [AIRFLOW-3090] Specify path of key file in log message URL: https://github.com/apache/incubator-airflow/pull/3921#issuecomment-422936030 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3921?src=pr&el=h1) Report > Merging [#3921](ht

[GitHub] odracci opened a new pull request #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-19 Thread GitBox
odracci opened a new pull request #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AI

[GitHub] odracci commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-09-19 Thread GitBox
odracci commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-422980969 I created #3922 as an alternative solution for this issue. /cc @dimberman @gerardo @Fokko --

[GitHub] dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-19 Thread GitBox
dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-422981515 @odracci Thank you for doing this!! Let me know when you want me to take a look :) -

[GitHub] dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose

2018-09-19 Thread GitBox
dimberman commented on issue #3797: [AIRFLOW-2952] Splits CI into k8s + docker-compose URL: https://github.com/apache/incubator-airflow/pull/3797#issuecomment-422982154 @odracci I like your solution better as a short-term fix (switching to k-d-c should ideally be done later when the build

[GitHub] odracci commented on issue #3770: [AIRFLOW-XXX] Fix Kubernetes operator with git-sync

2018-09-19 Thread GitBox
odracci commented on issue #3770: [AIRFLOW-XXX] Fix Kubernetes operator with git-sync URL: https://github.com/apache/incubator-airflow/pull/3770#issuecomment-422982400 Kubernetes integration tests are not working on the master branch, #3770 aim to fix the CI build. Once it is merged I wil

[GitHub] odracci edited a comment on issue #3770: [AIRFLOW-XXX] Fix Kubernetes operator with git-sync

2018-09-19 Thread GitBox
odracci edited a comment on issue #3770: [AIRFLOW-XXX] Fix Kubernetes operator with git-sync URL: https://github.com/apache/incubator-airflow/pull/3770#issuecomment-422982400 Kubernetes integration tests are not working on the master branch, #3922 aims to fix the CI build. Once it is merg

[GitHub] wrp opened a new pull request #3923: [AIRFLOW-3090] Make "No tasks to consider for execution." a debug msg

2018-09-19 Thread GitBox
wrp opened a new pull request #3923: [AIRFLOW-3090] Make "No tasks to consider for execution." a debug msg URL: https://github.com/apache/incubator-airflow/pull/3923 During normal operation, it is not necessary to see the message. This can only be useful when debugging an issue.

[GitHub] codecov-io commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-19 Thread GitBox
codecov-io commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-422988611 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3922?src=pr&el=h1) Report > Merging [#3922](https://codecov.io/gh/apach

[GitHub] dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-19 Thread GitBox
dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-422990887 @odracci Interesting! So just making sure I understand what's happening here: 1. You launch minikube + the kubernetes airflow deploymen

[GitHub] codecov-io commented on issue #3923: [AIRFLOW-3090] Make "No tasks to consider for execution." a debug msg

2018-09-19 Thread GitBox
codecov-io commented on issue #3923: [AIRFLOW-3090] Make "No tasks to consider for execution." a debug msg URL: https://github.com/apache/incubator-airflow/pull/3923#issuecomment-422991535 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3923?src=pr&el=h1) Report > Merg

[GitHub] XD-DENG opened a new pull request #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos

2018-09-19 Thread GitBox
XD-DENG opened a new pull request #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos URL: https://github.com/apache/incubator-airflow/pull/3924 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[GitHub] stale[bot] commented on issue #3039: update the password setter.

2018-09-19 Thread GitBox
stale[bot] commented on issue #3039: update the password setter. URL: https://github.com/apache/incubator-airflow/pull/3039#issuecomment-423014155 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Than

[GitHub] stale[bot] commented on issue #2490: User should be able to decide TemporaryDirectory root in a BashOperator

2018-09-19 Thread GitBox
stale[bot] commented on issue #2490: User should be able to decide TemporaryDirectory root in a BashOperator URL: https://github.com/apache/incubator-airflow/pull/2490#issuecomment-423014156 This issue has been automatically marked as stale because it has not had recent activity. It will

[GitHub] stale[bot] commented on issue #2666: Updating column header in table to match column.

2018-09-19 Thread GitBox
stale[bot] commented on issue #2666: Updating column header in table to match column. URL: https://github.com/apache/incubator-airflow/pull/2666#issuecomment-423014164 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further

[GitHub] stale[bot] commented on issue #2537: [AIRFLOW-419] Fix microsecond-precision datetime decode issue

2018-09-19 Thread GitBox
stale[bot] commented on issue #2537: [AIRFLOW-419] Fix microsecond-precision datetime decode issue URL: https://github.com/apache/incubator-airflow/pull/2537#issuecomment-423014166 This issue has been automatically marked as stale because it has not had recent activity. It will be closed

[GitHub] stale[bot] commented on issue #3283: allow TriggerDagRunOperator implementors to update dag id at runtime

2018-09-19 Thread GitBox
stale[bot] commented on issue #3283: allow TriggerDagRunOperator implementors to update dag id at runtime URL: https://github.com/apache/incubator-airflow/pull/3283#issuecomment-423014160 This issue has been automatically marked as stale because it has not had recent activity. It will be

[GitHub] stale[bot] commented on issue #1918: [AIRFLOW-668] Fix TypeError of run_command on python3

2018-09-19 Thread GitBox
stale[bot] commented on issue #1918: [AIRFLOW-668] Fix TypeError of run_command on python3 URL: https://github.com/apache/incubator-airflow/pull/1918#issuecomment-423014173 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no fu

[GitHub] stale[bot] commented on issue #1732: [AIRFLOW-428] Clean shutdown celery on SIGTERM

2018-09-19 Thread GitBox
stale[bot] commented on issue #1732: [AIRFLOW-428] Clean shutdown celery on SIGTERM URL: https://github.com/apache/incubator-airflow/pull/1732#issuecomment-423014151 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further a

[GitHub] codecov-io edited a comment on issue #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos

2018-09-19 Thread GitBox
codecov-io edited a comment on issue #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos URL: https://github.com/apache/incubator-airflow/pull/3924#issuecomment-423019364 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3924?src=pr&el=h1) Re

[GitHub] codecov-io commented on issue #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos

2018-09-19 Thread GitBox
codecov-io commented on issue #3924: [AIRFLOW-XXX] Fix a wrong sample bash command, a display issue & a few typos URL: https://github.com/apache/incubator-airflow/pull/3924#issuecomment-423019364 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3924?src=pr&el=h1) Report

[GitHub] zeninpalm opened a new pull request #3925: [AIRFLOW-3091]Deactivate inactive DAGs after collecting DagBag

2018-09-19 Thread GitBox
zeninpalm opened a new pull request #3925: [AIRFLOW-3091]Deactivate inactive DAGs after collecting DagBag URL: https://github.com/apache/incubator-airflow/pull/3925 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](https

[GitHub] codecov-io edited a comment on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset

2018-09-19 Thread GitBox
codecov-io edited a comment on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset URL: https://github.com/apache/incubator-airflow/pull/3876#issuecomment-420583503 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3876?src=pr&el=h

[GitHub] seelmann commented on a change in pull request #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-19 Thread GitBox
seelmann commented on a change in pull request #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596#discussion_r219044092 ## File path: airflow/models.py ## @@ -1744,6 +1749,29 @@ def dry_run(self): self.

[GitHub] seelmann commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-19 Thread GitBox
seelmann commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596#issuecomment-423052692 @feng-tao I replied and resolved all your questions and comments, PTAL. Currently I added additional commits on top, when all

[GitHub] codecov-io edited a comment on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset URL: https://github.com/apache/incubator-airflow/pull/3876#issuecomment-420583503 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3876?src=pr&el=h

[GitHub] odracci commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
odracci commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#discussion_r219071347 ## File path: scripts/ci/kubernetes/kube/configmaps.yaml ## @@ -27,7 +27,7 @@ data: logging_level = INF

[GitHub] odracci commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
odracci commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-423095904 @dimberman correct, the key is to set `network_mode: host` in the docker-compose file. ---

[GitHub] ashb commented on issue #3925: [AIRFLOW-3091]Deactivate inactive DAGs after collecting DagBag

2018-09-20 Thread GitBox
ashb commented on issue #3925: [AIRFLOW-3091]Deactivate inactive DAGs after collecting DagBag URL: https://github.com/apache/incubator-airflow/pull/3925#issuecomment-423104714 What testing of this have you performed? I'm not sure the fix is this simple - due to all the processes, an

[GitHub] xnuinside commented on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset

2018-09-20 Thread GitBox
xnuinside commented on issue #3876: [AIRFLOW-2887] Add to BigQueryBaseCursor methods for insert (create empty) dataset URL: https://github.com/apache/incubator-airflow/pull/3876#issuecomment-423126280 @kaxil , fixed, sorry for taking a long time ---

[GitHub] ashb commented on issue #2598: [AIRFLOW-1595] Change to construct sqlite_hook from connection schema

2018-09-20 Thread GitBox
ashb commented on issue #2598: [AIRFLOW-1595] Change to construct sqlite_hook from connection schema URL: https://github.com/apache/incubator-airflow/pull/2598#issuecomment-423131504 To resurrect an old PR: This PR might make sense, but as it is it would break any existing connections tha

[GitHub] ron819 commented on issue #2113: [AIRFLOW-920] Allow marking tasks in zoomed in subdags

2018-09-20 Thread GitBox
ron819 commented on issue #2113: [AIRFLOW-920] Allow marking tasks in zoomed in subdags URL: https://github.com/apache/incubator-airflow/pull/2113#issuecomment-423192710 @bolkedebruin are you still working on this? This is a

[GitHub] ron819 commented on issue #2776: [AIRFLOW-1793] fix DockerOperator using docker_conn_id

2018-09-20 Thread GitBox
ron819 commented on issue #2776: [AIRFLOW-1793] fix DockerOperator using docker_conn_id URL: https://github.com/apache/incubator-airflow/pull/2776#issuecomment-423196176 @m1racoli There are conflicts to resolve This is an au

[GitHub] codecov-io edited a comment on issue #3138: [AIRFLOW-2221] Create DagFetcher abstraction

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3138: [AIRFLOW-2221] Create DagFetcher abstraction URL: https://github.com/apache/incubator-airflow/pull/3138#issuecomment-373965660 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3138?src=pr&el=h1) Report > Merging [#3138](https

[GitHub] codecov-io edited a comment on issue #3741: [AIRFLOW-1368] Add auto_remove for DockerOperator

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3741: [AIRFLOW-1368] Add auto_remove for DockerOperator URL: https://github.com/apache/incubator-airflow/pull/3741#issuecomment-412492251 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3741?src=pr&el=h1) Report > Merging [#3741](

[GitHub] aaronmak opened a new pull request #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies

2018-09-20 Thread GitBox
aaronmak opened a new pull request #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies URL: https://github.com/apache/incubator-airflow/pull/3926 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apac

[GitHub] aaronmak closed pull request #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies

2018-09-20 Thread GitBox
aaronmak closed pull request #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies URL: https://github.com/apache/incubator-airflow/pull/3926 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

[GitHub] aaronmak opened a new pull request #3927: [AIRFLOW-XXX] Add 90 Seconds to companies

2018-09-20 Thread GitBox
aaronmak opened a new pull request #3927: [AIRFLOW-XXX] Add 90 Seconds to companies URL: https://github.com/apache/incubator-airflow/pull/3927 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/j

[GitHub] smith-m commented on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator.

2018-09-20 Thread GitBox
smith-m commented on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator. URL: https://github.com/apache/incubator-airflow/pull/3908#issuecomment-423231025 @jmcarp JIRA marked this issue (Also Airflow-3036? GCP CloudSQL/mysql configuration issues with 1.10) as comp

[GitHub] smith-m edited a comment on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator.

2018-09-20 Thread GitBox
smith-m edited a comment on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator. URL: https://github.com/apache/incubator-airflow/pull/3908#issuecomment-423231025 @jmcarp JIRA marked this issue (Also Airflow-3036? GCP CloudSQL/mysql configuration issues with 1.10)

[GitHub] smith-m edited a comment on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator.

2018-09-20 Thread GitBox
smith-m edited a comment on issue #3908: [AIRFLOW-3036] Add relevant ECS options to ECS operator. URL: https://github.com/apache/incubator-airflow/pull/3908#issuecomment-423231025 @jmcarp JIRA marked this issue (Also Airflow-3036? GCP CloudSQL/mysql configuration issues with 1.10)

[GitHub] r39132 commented on issue #3927: [AIRFLOW-XXX] Add 90 Seconds to companies

2018-09-20 Thread GitBox
r39132 commented on issue #3927: [AIRFLOW-XXX] Add 90 Seconds to companies URL: https://github.com/apache/incubator-airflow/pull/3927#issuecomment-423232676 +1 Welcome to the Apache Airflow community! This is an automated mes

[GitHub] r39132 edited a comment on issue #3927: [AIRFLOW-XXX] Add 90 Seconds to companies

2018-09-20 Thread GitBox
r39132 edited a comment on issue #3927: [AIRFLOW-XXX] Add 90 Seconds to companies URL: https://github.com/apache/incubator-airflow/pull/3927#issuecomment-423232676 +1 Welcome to the Apache Airflow community @aaronmak ! This

[GitHub] r39132 closed pull request #3927: [AIRFLOW-XXX] Add 90 Seconds to companies

2018-09-20 Thread GitBox
r39132 closed pull request #3927: [AIRFLOW-XXX] Add 90 Seconds to companies URL: https://github.com/apache/incubator-airflow/pull/3927 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 f

[GitHub] r39132 opened a new pull request #3928: [AIRFLOW-3096] Reduce DaysUntilStale for probot/stale

2018-09-20 Thread GitBox
r39132 opened a new pull request #3928: [AIRFLOW-3096] Reduce DaysUntilStale for probot/stale URL: https://github.com/apache/incubator-airflow/pull/3928 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.ap

[GitHub] r39132 closed pull request #3928: [AIRFLOW-3096] Reduce DaysUntilStale for probot/stale

2018-09-20 Thread GitBox
r39132 closed pull request #3928: [AIRFLOW-3096] Reduce DaysUntilStale for probot/stale URL: https://github.com/apache/incubator-airflow/pull/3928 This is an automated message from the Apache Git Service. To respond to the m

[GitHub] dimberman commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
dimberman commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#discussion_r219228533 ## File path: scripts/ci/kubernetes/kube/configmaps.yaml ## @@ -27,7 +27,7 @@ data: logging_level = I

[GitHub] codecov-io commented on issue #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies

2018-09-20 Thread GitBox
codecov-io commented on issue #3926: [AIRFLOW-XXX] add 90 Seconds to list of companies URL: https://github.com/apache/incubator-airflow/pull/3926#issuecomment-423246639 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3926?src=pr&el=h1) Report > Merging [#3926](https:/

[GitHub] r39132 opened a new pull request #3929: [AIRFLOW-3096] Further reduce DaysUntilStale for probo/stale

2018-09-20 Thread GitBox
r39132 opened a new pull request #3929: [AIRFLOW-3096] Further reduce DaysUntilStale for probo/stale URL: https://github.com/apache/incubator-airflow/pull/3929 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://is

[GitHub] r39132 closed pull request #3929: [AIRFLOW-3096] Further reduce DaysUntilStale for probo/stale

2018-09-20 Thread GitBox
r39132 closed pull request #3929: [AIRFLOW-3096] Further reduce DaysUntilStale for probo/stale URL: https://github.com/apache/incubator-airflow/pull/3929 This is an automated message from the Apache Git Service. To respond t

[GitHub] dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
dimberman commented on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-423289297 cc: @ashb @kaxil any help on merging this would be awesome since it's blocking all other k8s work :). --

[GitHub] codecov-io commented on issue #3894: WIP: [AIRFLOW-3055] add get_dataset and get_datasets_list to bigquery_hook

2018-09-20 Thread GitBox
codecov-io commented on issue #3894: WIP: [AIRFLOW-3055] add get_dataset and get_datasets_list to bigquery_hook URL: https://github.com/apache/incubator-airflow/pull/3894#issuecomment-423291432 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3894?src=pr&el=h1) Report >

[GitHub] jgao54 closed pull request #3913: [AIRFLOW-3072] Assign permission get_logs_with_metadata to viewer role

2018-09-20 Thread GitBox
jgao54 closed pull request #3913: [AIRFLOW-3072] Assign permission get_logs_with_metadata to viewer role URL: https://github.com/apache/incubator-airflow/pull/3913 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] yrqls21 commented on issue #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug

2018-09-20 Thread GitBox
yrqls21 commented on issue #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug URL: https://github.com/apache/incubator-airflow/pull/3920#issuecomment-423296311 I'm +1 for this change This is an automated messa

[GitHub] xnuinside commented on issue #3894: [AIRFLOW-3055] add get_dataset and get_datasets_list to bigquery_hook

2018-09-20 Thread GitBox
xnuinside commented on issue #3894: [AIRFLOW-3055] add get_dataset and get_datasets_list to bigquery_hook URL: https://github.com/apache/incubator-airflow/pull/3894#issuecomment-423303142 @kaxil, wanna to add methods to hook - get_dataset and get_datasets_list, please, review --

[GitHub] Fokko closed pull request #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug

2018-09-20 Thread GitBox
Fokko closed pull request #3920: [AIRFLOW-3090] Demote dag start/stop log messages to debug URL: https://github.com/apache/incubator-airflow/pull/3920 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

[GitHub] Fokko closed pull request #3298: [AIRFLOW-2407] Use feature detection for reload()

2018-09-20 Thread GitBox
Fokko closed pull request #3298: [AIRFLOW-2407] Use feature detection for reload() URL: https://github.com/apache/incubator-airflow/pull/3298 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 thi

[GitHub] Fokko commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
Fokko commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#discussion_r219303693 ## File path: tests/contrib/minikube/test_kubernetes_executor.py ## @@ -25,18 +26,25 @@ import re try: -

[GitHub] odracci commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
odracci commented on a change in pull request #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#discussion_r219308984 ## File path: tests/contrib/minikube/test_kubernetes_executor.py ## @@ -25,18 +26,25 @@ import re try: -

[GitHub] codecov-io edited a comment on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-422988611 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3922?src=pr&el=h1) Report > Merging [#3922](https://codecov.io/g

[GitHub] codecov-io edited a comment on issue #3922: AIRFLOW-2952 Fix Kubernetes CI

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3922: AIRFLOW-2952 Fix Kubernetes CI URL: https://github.com/apache/incubator-airflow/pull/3922#issuecomment-422988611 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3922?src=pr&el=h1) Report > Merging [#3922](https://codecov.io/g

[GitHub] gwax commented on issue #3772: [AIRFLOW-2918] Fix Flake8 violations

2018-09-20 Thread GitBox
gwax commented on issue #3772: [AIRFLOW-2918] Fix Flake8 violations URL: https://github.com/apache/incubator-airflow/pull/3772#issuecomment-423363715 👍 for black and strong support for using `pre-commit` as well This is an au

[GitHub] feng-tao commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-20 Thread GitBox
feng-tao commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596#issuecomment-423371851 Will take a look tonight. This is an automated message from

[GitHub] jcao219 opened a new pull request #3930: [AIRFLOW-2548] Output plugin import errors to web UI

2018-09-20 Thread GitBox
jcao219 opened a new pull request #3930: [AIRFLOW-2548] Output plugin import errors to web UI URL: https://github.com/apache/incubator-airflow/pull/3930 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.ap

[GitHub] yrqls21 commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop

2018-09-20 Thread GitBox
yrqls21 commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop URL: https://github.com/apache/incubator-airflow/pull/3873#discussion_r219360728 ## File path: airflow/utils/dag_processing.py ## @@ -22,17 +22,39 @@ from __f

[GitHub] XD-DENG commented on issue #3903: [AIRFLOW-3067] Display www_rbac Flask flash msg properly

2018-09-20 Thread GitBox
XD-DENG commented on issue #3903: [AIRFLOW-3067] Display www_rbac Flask flash msg properly URL: https://github.com/apache/incubator-airflow/pull/3903#issuecomment-423383607 Hi @feng-tao , a gentle ping. May you take another look? @ashb , I believe this is something worth pushing to

[GitHub] ubermen commented on issue #3885: [AIRFLOW-3001] Add index 'ti_dag_date' to taskinstance

2018-09-20 Thread GitBox
ubermen commented on issue #3885: [AIRFLOW-3001] Add index 'ti_dag_date' to taskinstance URL: https://github.com/apache/incubator-airflow/pull/3885#issuecomment-423393327 I've resolved some CI issues. Please confirm or correct any problems. Thanks :) -

[GitHub] codecov-io edited a comment on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-20 Thread GitBox
codecov-io edited a comment on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596#issuecomment-422536512 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3596?src=pr&el=h1) Report > Merging [#3596](htt

[GitHub] feng-tao commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-20 Thread GitBox
feng-tao commented on issue #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596#issuecomment-423413759 Thanks @seelmann . LGTM. This is an automated message from t

[GitHub] feng-tao closed pull request #3596: [AIRFLOW-2747] Explicit re-schedule of sensors

2018-09-20 Thread GitBox
feng-tao closed pull request #3596: [AIRFLOW-2747] Explicit re-schedule of sensors URL: https://github.com/apache/incubator-airflow/pull/3596 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 thi

[GitHub] dlamblin commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop

2018-09-20 Thread GitBox
dlamblin commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop URL: https://github.com/apache/incubator-airflow/pull/3873#discussion_r219384327 ## File path: airflow/utils/dag_processing.py ## @@ -308,6 +369,249 @@ def fi

[GitHub] dlamblin commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop

2018-09-20 Thread GitBox
dlamblin commented on a change in pull request #3873: [Airflow-2760] Decouple DAG parsing loop from scheduler loop URL: https://github.com/apache/incubator-airflow/pull/3873#discussion_r219384327 ## File path: airflow/utils/dag_processing.py ## @@ -308,6 +369,249 @@ def fi

[GitHub] feng-tao commented on issue #3903: [AIRFLOW-3067] Display www_rbac Flask flash msg properly

2018-09-20 Thread GitBox
feng-tao commented on issue #3903: [AIRFLOW-3067] Display www_rbac Flask flash msg properly URL: https://github.com/apache/incubator-airflow/pull/3903#issuecomment-423414186 Overall LGTM. Thanks @XD-DENG . Could you attach the latest UI with your pr? If the UI figure looks good, we are go

<    1   2   3   4   5   6   7   8   9   10   >