[GitHub] [airflow-site] kgabryje commented on a change in pull request #183: Content/commiters fix

2019-11-19 Thread GitBox
kgabryje commented on a change in pull request #183: Content/commiters fix URL: https://github.com/apache/airflow-site/pull/183#discussion_r347774990 ## File path: landing-pages/site/assets/scss/_list-boxes.scss ## @@ -39,11 +40,12 @@ $card-margin: 20px; .list-item {

[GitHub] [airflow-site] kgabryje commented on a change in pull request #183: Content/commiters fix

2019-11-19 Thread GitBox
kgabryje commented on a change in pull request #183: Content/commiters fix URL: https://github.com/apache/airflow-site/pull/183#discussion_r347776976 ## File path: landing-pages/site/assets/scss/_list-boxes.scss ## @@ -63,9 +63,11 @@ $card-margin: 20px; border-radius: 5p

[GitHub] [airflow-site] kgabryje commented on a change in pull request #183: Content/commiters fix

2019-11-19 Thread GitBox
kgabryje commented on a change in pull request #183: Content/commiters fix URL: https://github.com/apache/airflow-site/pull/183#discussion_r347782904 ## File path: landing-pages/site/assets/scss/_list-boxes.scss ## @@ -39,11 +40,12 @@ $card-margin: 20px; .list-item {

[GitHub] [airflow] potiuk commented on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6010][AIRFLOW-6004]

2019-11-19 Thread GitBox
potiuk commented on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6010][AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6599#issuecomment-555401489 Without those changes, we have cyclic dependencies in this PR. --

[GitHub] [airflow] potiuk commented on issue #6595: [AIRFLOW-6002] Drop support for python 3.5

2019-11-19 Thread GitBox
potiuk commented on issue #6595: [AIRFLOW-6002] Drop support for python 3.5 URL: https://github.com/apache/airflow/pull/6595#issuecomment-555401836 and 2.7 :) This is an automated message from the Apache Git Service. To respon

[GitHub] [airflow] potiuk commented on issue #6595: [AIRFLOW-6002] Drop support for python 3.5

2019-11-19 Thread GitBox
potiuk commented on issue #6595: [AIRFLOW-6002] Drop support for python 3.5 URL: https://github.com/apache/airflow/pull/6595#issuecomment-555402601 One small problem with it - this is for the "released" version and is taken directly from the PyPI. Latest released version supports 2.7, 3.5,

[GitHub] [airflow-site] mik-laj merged pull request #183: Content/commiters fix

2019-11-19 Thread GitBox
mik-laj merged pull request #183: Content/commiters fix URL: https://github.com/apache/airflow-site/pull/183 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[airflow-site] branch aip-11 updated (f4fca20 -> af3322f)

2019-11-19 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a change to branch aip-11 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. from f4fca20 Make scrollbar in docs sidenav always visible (#177) add af3322f Update commiters list an

[GitHub] [airflow] codecov-io edited a comment on issue #6380: [AIRFLOW-3632] Allow replace_microseconds in trigger_dag REST request

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6380: [AIRFLOW-3632] Allow replace_microseconds in trigger_dag REST request URL: https://github.com/apache/airflow/pull/6380#issuecomment-544885909 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6380?src=pr&el=h1) Report > Merging [#6380](ht

[GitHub] [airflow-site] kgabryje opened a new pull request #184: Content/archetypes

2019-11-19 Thread GitBox
kgabryje opened a new pull request #184: Content/archetypes URL: https://github.com/apache/airflow-site/pull/184 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-555427411 @potiuk Like this? This is an automate

[GitHub] [airflow] simonohanlon101 opened a new pull request #6605: TokenAnalyst readme update

2019-11-19 Thread GitBox
simonohanlon101 opened a new pull request #6605: TokenAnalyst readme update URL: https://github.com/apache/airflow/pull/6605 Make sure you have checked _all_ steps below. ### Description - [x] Here are some details about my PR, including screenshots of any UI changes: Addin

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r347854161 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_FO

[jira] [Created] (AIRFLOW-6014) Kubernetes executor - handle preempted deleted pods - queued tasks

2019-11-19 Thread afusr (Jira)
afusr created AIRFLOW-6014: -- Summary: Kubernetes executor - handle preempted deleted pods - queued tasks Key: AIRFLOW-6014 URL: https://issues.apache.org/jira/browse/AIRFLOW-6014 Project: Apache Airflow

[jira] [Updated] (AIRFLOW-6014) Kubernetes executor - handle preempted deleted pods - queued tasks

2019-11-19 Thread afusr (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] afusr updated AIRFLOW-6014: --- Description: We have encountered an issue whereby when using the kubernetes executor, and using autoscalin

[GitHub] [airflow] potiuk commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
potiuk commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-555452928 @feluelle. Exactly ! I think the only thing missing is to restore the connection after the tests. It shou

[GitHub] [airflow] atrbgithub opened a new pull request #6606: [AIRFLOW-6014] - handle pods which are preempted and deleted by kuber…

2019-11-19 Thread GitBox
atrbgithub opened a new pull request #6606: [AIRFLOW-6014] - handle pods which are preempted and deleted by kuber… URL: https://github.com/apache/airflow/pull/6606 …netes but not restarted Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the

[jira] [Commented] (AIRFLOW-6014) Kubernetes executor - handle preempted deleted pods - queued tasks

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977386#comment-16977386 ] ASF GitHub Bot commented on AIRFLOW-6014: - atrbgithub commented on pull request

[GitHub] [airflow] ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-19 Thread GitBox
ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r347861744 ## File path: tests/dags/test_subdag.py ## @@ -24,7 +24,7 @@

[GitHub] [airflow] feluelle commented on issue #6575: [WIP][AIRFLOW-5921] Add bulk_load_custom to MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6575: [WIP][AIRFLOW-5921] Add bulk_load_custom to MySqlHook URL: https://github.com/apache/airflow/pull/6575#issuecomment-555457144 @OmerJog PTAL This is an automated message from the Apache Git S

[GitHub] [airflow] ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-19 Thread GitBox
ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r347861744 ## File path: tests/dags/test_subdag.py ## @@ -24,7 +24,7 @@

[GitHub] [airflow] ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-19 Thread GitBox
ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r347861744 ## File path: tests/dags/test_subdag.py ## @@ -24,7 +24,7 @@

[jira] [Commented] (AIRFLOW-6014) Kubernetes executor - handle preempted deleted pods - queued tasks

2019-11-19 Thread afusr (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977392#comment-16977392 ] afusr commented on AIRFLOW-6014: The following PR has been raised as a temp to resolve

[GitHub] [airflow] ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010]

2019-11-19 Thread GitBox
ashb commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports. Depends on [AIRFLOW-6010] URL: https://github.com/apache/airflow/pull/6596#discussion_r347861744 ## File path: tests/dags/test_subdag.py ## @@ -24,7 +24,7 @@

[GitHub] [airflow-site] kgabryje opened a new pull request #185: add meta description to head

2019-11-19 Thread GitBox
kgabryje opened a new pull request #185: add meta description to head URL: https://github.com/apache/airflow-site/pull/185 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Commented] (AIRFLOW-6014) Kubernetes executor - handle preempted deleted pods - queued tasks

2019-11-19 Thread Daniel Imberman (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977405#comment-16977405 ] Daniel Imberman commented on AIRFLOW-6014: -- Hmmm this is an interesting on

[GitHub] [airflow] codecov-io edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-553564200 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6576?src=pr&el=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-553564200 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6576?src=pr&el=h1) Report > Merging

[GitHub] [airflow-site] mik-laj merged pull request #185: add meta description to head

2019-11-19 Thread GitBox
mik-laj merged pull request #185: add meta description to head URL: https://github.com/apache/airflow-site/pull/185 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[airflow-site] branch aip-11 updated (af3322f -> 4ab2599)

2019-11-19 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a change to branch aip-11 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. from af3322f Update commiters list and fix spacing (#183) add 4ab2599 Add meta description to head (#1

[GitHub] [airflow] OmerJog commented on a change in pull request #6575: [AIRFLOW-5921] Add bulk_load_custom to MySqlHook

2019-11-19 Thread GitBox
OmerJog commented on a change in pull request #6575: [AIRFLOW-5921] Add bulk_load_custom to MySqlHook URL: https://github.com/apache/airflow/pull/6575#discussion_r347927210 ## File path: tests/hooks/test_mysql_hook.py ## @@ -236,3 +236,21 @@ def test_bulk_dump(self):

[GitHub] [airflow] codecov-io edited a comment on issue #6575: [AIRFLOW-5921] Add bulk_load_custom to MySqlHook

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6575: [AIRFLOW-5921] Add bulk_load_custom to MySqlHook URL: https://github.com/apache/airflow/pull/6575#issuecomment-553557551 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6575?src=pr&el=h1) Report > Merging [#6575](https://codecov.io/gh/a

[jira] [Commented] (AIRFLOW-5462) Google Authentication redirection page crashes with KeyError: 'login'

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5462?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977517#comment-16977517 ] ASF GitHub Bot commented on AIRFLOW-5462: - paulvic commented on pull request #6

[GitHub] [airflow] paulvic opened a new pull request #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-19 Thread GitBox
paulvic opened a new pull request #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607 Same fix as was required for Superset (https://github.com/apache/incubator-superset/issues/7739) Make sure you have checked _all_ steps

[GitHub] [airflow] ashb commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-19 Thread GitBox
ashb commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607#issuecomment-34433 All in favour of this change, but have you checked the rest of the UI? Has anything else around filtering or searching broken or

[GitHub] [airflow] feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-34894 @potiuk hmm? :) This is an automated m

[GitHub] [airflow] ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-35547 Do we need both options? When might someone want one over the other? --

[GitHub] [airflow] ashb commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
ashb commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r347960018 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_FOLD

[GitHub] [airflow] feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-36862 That is currently the only one I experienced. ``` .. note:: By default it connects to the

[GitHub] [airflow] feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-36862 That is currently the only one I experienced. ``` .. note:: By default it connects

[GitHub] [airflow] feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-36862 That is currently the only one I experienced. ``` .. note:: By default it connects

[GitHub] [airflow] feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-36862 That is currently the only one I experienced. ``` .. note:: By default it connects

[GitHub] [airflow] feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-36862 That is currently the only one I experienced. ``` .. note:: By default it connects

[GitHub] [airflow] codecov-io edited a comment on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607#issuecomment-42551 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6607?src=pr&el=h1) Report > Merging [#6607](https://codeco

[GitHub] [airflow] codecov-io commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue

2019-11-19 Thread GitBox
codecov-io commented on issue #6607: [AIRFLOW-5462] Bump FAB to 2.2.0 to fix OAuth login issue URL: https://github.com/apache/airflow/pull/6607#issuecomment-42551 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6607?src=pr&el=h1) Report > Merging [#6607](https://codecov.io/gh

[GitHub] [airflow] feluelle commented on issue #6252: [AIRFLOW-4676] Make airflow/macros Pylint compatible

2019-11-19 Thread GitBox
feluelle commented on issue #6252: [AIRFLOW-4676] Make airflow/macros Pylint compatible URL: https://github.com/apache/airflow/pull/6252#issuecomment-51965 Hey @arkadiuszbach can you rebase so I can merge it? :) This is a

[GitHub] [airflow] feluelle commented on a change in pull request #5755: [AIRFLOW-4568] Add unallowed_states to ExternalTaskSensor

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #5755: [AIRFLOW-4568] Add unallowed_states to ExternalTaskSensor URL: https://github.com/apache/airflow/pull/5755#discussion_r347982988 ## File path: airflow/sensors/external_task_sensor.py ## @@ -40,6 +40,10 @@ class External

[GitHub] [airflow] ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-59434 > Because you cannot use pure ssl without any cert options in mysqlclient 😱 ---

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r347993579 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_FO

[GitHub] [airflow] feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r348010222 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_

[GitHub] [airflow] feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r348012438 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_

[GitHub] [airflow] ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
ashb commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-75421 Is there a reson to keep the old one, or would just switching to mysql-connector-python be sensible? (Not v

[GitHub] [airflow] feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly.

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6601: [AIRFLOW-6010] Do not import airflow settings directly. URL: https://github.com/apache/airflow/pull/6601#discussion_r348010222 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log -DAGS_

[GitHub] [airflow] feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-78907 We are using `mysql-connector-python` in our airflow-plugins. But I would stay with `mysqlclient` if ss

[GitHub] [airflow] feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle edited a comment on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-78907 We are using `mysql-connector-python` in our airflow-plugins. But I would stay with `mysqlclient

[jira] [Commented] (AIRFLOW-6001) Lazy load CLI commands

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977597#comment-16977597 ] ASF GitHub Bot commented on AIRFLOW-6001: - mik-laj commented on pull request #6

[GitHub] [airflow] mik-laj merged pull request #6594: [AIRFLOW-6001] Lazy load CLI commands

2019-11-19 Thread GitBox
mik-laj merged pull request #6594: [AIRFLOW-6001] Lazy load CLI commands URL: https://github.com/apache/airflow/pull/6594 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [airflow] feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook

2019-11-19 Thread GitBox
feluelle commented on issue #6576: [AIRFLOW-5922] Add option to specify the mysql client library used in MySqlHook URL: https://github.com/apache/airflow/pull/6576#issuecomment-81059 > > Because you cannot use pure ssl without any cert options in mysqlclient > > 😱 😄 Normal

[jira] [Commented] (AIRFLOW-6001) Lazy load CLI commands

2019-11-19 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977599#comment-16977599 ] ASF subversion and git services commented on AIRFLOW-6001: -- Co

[jira] [Commented] (AIRFLOW-5060) Add support of CatalogId to AwsGlueCatalogHook

2019-11-19 Thread jack (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977628#comment-16977628 ] jack commented on AIRFLOW-5060: --- [~ash] any thoughts? > Add support of CatalogId to AwsG

[GitHub] [airflow] feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator URL: https://github.com/apache/airflow/pull/5900#discussion_r348048611 ## File path: airflow/contrib/operators/awsbatch_operator.py ## @@ -62,6 +62,10 @@ class AWS

[GitHub] [airflow] feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator URL: https://github.com/apache/airflow/pull/5900#discussion_r348049778 ## File path: airflow/contrib/operators/awsbatch_operator.py ## @@ -62,6 +62,10 @@ class AWS

[GitHub] [airflow] feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator URL: https://github.com/apache/airflow/pull/5900#discussion_r348048825 ## File path: airflow/contrib/operators/awsbatch_operator.py ## @@ -62,6 +62,10 @@ class AWS

[GitHub] [airflow] feluelle commented on issue #5900: [AIRFLOW-5313] Add params support for awsbatch_operator

2019-11-19 Thread GitBox
feluelle commented on issue #5900: [AIRFLOW-5313] Add params support for awsbatch_operator URL: https://github.com/apache/airflow/pull/5900#issuecomment-555605749 Please also fix the conflicts and rebase onto master. Thanks 👍 :) -

[GitHub] [airflow] feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator URL: https://github.com/apache/airflow/pull/6469#discussion_r348076635 ## File path: airflow/contrib/operators/s3_to_snowflake_operator.py ## @@ -0,0 +1,110 @@ +# -*- coding: utf-

[GitHub] [airflow] feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator URL: https://github.com/apache/airflow/pull/6469#discussion_r348077863 ## File path: tests/contrib/operators/test_s3_to_snowflake_operator.py ## @@ -0,0 +1,69 @@ +# -*- coding: ut

[GitHub] [airflow] feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator

2019-11-19 Thread GitBox
feluelle commented on a change in pull request #6469: [AIRFLOW-5816] S3 to snowflake operator URL: https://github.com/apache/airflow/pull/6469#discussion_r348055541 ## File path: airflow/contrib/operators/s3_to_snowflake_operator.py ## @@ -0,0 +1,110 @@ +# -*- coding: utf-

[GitHub] [airflow] amichai07 commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run

2019-11-19 Thread GitBox
amichai07 commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run URL: https://github.com/apache/airflow/pull/4751#discussion_r348077144 ## File path: airflow/jobs/scheduler_job.py ## @@ -721,25 +721,14 @@ def _process_task_

[GitHub] [airflow] amichai07 commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run

2019-11-19 Thread GitBox
amichai07 commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run URL: https://github.com/apache/airflow/pull/4751#discussion_r348080871 ## File path: airflow/jobs/scheduler_job.py ## @@ -721,25 +721,14 @@ def _process_task_

[jira] [Resolved] (AIRFLOW-4145) Allow RBAC roles permissions , VM to be overridable

2019-11-19 Thread Ash Berlin-Taylor (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor resolved AIRFLOW-4145. Fix Version/s: 1.10.7 Resolution: Fixed > Allow RBAC roles permissions , V

[GitHub] [airflow] jrowen commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator

2019-11-19 Thread GitBox
jrowen commented on a change in pull request #5900: [AIRFLOW-5313] Add params support for awsbatch_operator URL: https://github.com/apache/airflow/pull/5900#discussion_r348098115 ## File path: airflow/contrib/operators/awsbatch_operator.py ## @@ -62,6 +62,10 @@ class AWSBa

[jira] [Created] (AIRFLOW-6015) Celery needed even if LocalExecutor is configured

2019-11-19 Thread Mathieu Cinquin (Jira)
Mathieu Cinquin created AIRFLOW-6015: Summary: Celery needed even if LocalExecutor is configured Key: AIRFLOW-6015 URL: https://issues.apache.org/jira/browse/AIRFLOW-6015 Project: Apache Airflow

[jira] [Assigned] (AIRFLOW-5950) Change import paths for "apache/cassandra" modules

2019-11-19 Thread Rich Dean (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5950?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Rich Dean reassigned AIRFLOW-5950: -- Assignee: Rich Dean > Change import paths for "apache/cassandra" modules > -

[jira] [Closed] (AIRFLOW-6015) Celery needed even if LocalExecutor is configured

2019-11-19 Thread Ash Berlin-Taylor (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6015?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ash Berlin-Taylor closed AIRFLOW-6015. -- Resolution: Not A Problem {{airflow worker}} is for running Celery Workers -- when yo

[GitHub] [airflow] ashb commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run

2019-11-19 Thread GitBox
ashb commented on a change in pull request #4751: [AIRFLOW-3607] collected trigger rule dep check per dag run URL: https://github.com/apache/airflow/pull/4751#discussion_r348126662 ## File path: airflow/jobs/scheduler_job.py ## @@ -721,25 +721,14 @@ def _process_task_insta

[jira] [Assigned] (AIRFLOW-5780) Migrate AWS Glue to /providers/aws [AIP-21]

2019-11-19 Thread Sergio Kef (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sergio Kef reassigned AIRFLOW-5780: --- Assignee: Sergio Kef > Migrate AWS Glue to /providers/aws [AIP-21] > -

[GitHub] [airflow] serkef opened a new pull request #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers

2019-11-19 Thread GitBox
serkef opened a new pull request #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers URL: https://github.com/apache/airflow/pull/6608 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/

[jira] [Commented] (AIRFLOW-5780) Migrate AWS Glue to /providers/aws [AIP-21]

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5780?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977822#comment-16977822 ] ASF GitHub Bot commented on AIRFLOW-5780: - serkef commented on pull request #66

[jira] [Created] (AIRFLOW-6016) Remove redundant config option `remote_logging`

2019-11-19 Thread Sergio Kef (Jira)
Sergio Kef created AIRFLOW-6016: --- Summary: Remove redundant config option `remote_logging` Key: AIRFLOW-6016 URL: https://issues.apache.org/jira/browse/AIRFLOW-6016 Project: Apache Airflow Issu

[GitHub] [airflow] Fokko commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations

2019-11-19 Thread GitBox
Fokko commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations URL: https://github.com/apache/airflow/pull/6447#issuecomment-555714460 Sorry for the late response, I'm extremely busy right now. Right now there are three heads on v1-10-stable: ``` MacBook-Pro-van-Fo

[GitHub] [airflow] Fokko commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations

2019-11-19 Thread GitBox
Fokko commented on issue #6447: [AIRFLOW-5792] Straighten out the migrations URL: https://github.com/apache/airflow/pull/6447#issuecomment-555715574 If we want to do this the clean way, then we have to figure out if the heads are already part of a release. If not, then remove the branches i

[GitHub] [airflow] ratb3rt opened a new pull request #6609: [AIRFLOW-5950] AIP-21 Change import paths for "apache/cassandra" modules

2019-11-19 Thread GitBox
ratb3rt opened a new pull request #6609: [AIRFLOW-5950] AIP-21 Change import paths for "apache/cassandra" modules URL: https://github.com/apache/airflow/pull/6609 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow Jira](https:/

[jira] [Commented] (AIRFLOW-5950) Change import paths for "apache/cassandra" modules

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5950?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977839#comment-16977839 ] ASF GitHub Bot commented on AIRFLOW-5950: - ratb3rt commented on pull request #6

[GitHub] [airflow] codecov-io commented on issue #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers

2019-11-19 Thread GitBox
codecov-io commented on issue #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers URL: https://github.com/apache/airflow/pull/6608#issuecomment-555726762 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6608?src=pr&el=h1) Report > Merging [#6608](https://codecov.io/gh/ap

[GitHub] [airflow] codecov-io edited a comment on issue #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6608: [AIRFLOW-5780] Move AWS Glue from contrib to providers URL: https://github.com/apache/airflow/pull/6608#issuecomment-555726762 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6608?src=pr&el=h1) Report > Merging [#6608](https://codecov.i

[GitHub] [airflow] serkef opened a new pull request #6610: [AIRFLOW-6016] Remove redundant setting `remote_logging`

2019-11-19 Thread GitBox
serkef opened a new pull request #6610: [AIRFLOW-6016] Remove redundant setting `remote_logging` URL: https://github.com/apache/airflow/pull/6610 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.or

[jira] [Commented] (AIRFLOW-6016) Remove redundant config option `remote_logging`

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977848#comment-16977848 ] ASF GitHub Bot commented on AIRFLOW-6016: - serkef commented on pull request #66

[jira] [Updated] (AIRFLOW-6010) The "from airflow import settings" is used far too often

2019-11-19 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6010?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-6010: -- Description: Context for DAG context management was loaded from settings and it causede some

[jira] [Updated] (AIRFLOW-6010) The "from airflow import settings" is used far too often

2019-11-19 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6010?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-6010: -- Description: Context for DAG context management was loaded from settings and it caused some

[GitHub] [airflow] potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Settings are not used for DAG context management

2019-11-19 Thread GitBox
potiuk commented on a change in pull request #6601: [AIRFLOW-6010] Settings are not used for DAG context management URL: https://github.com/apache/airflow/pull/6601#discussion_r348189503 ## File path: airflow/bin/cli.py ## @@ -70,10 +70,18 @@ LOG = LoggingMixin().log

[airflow-site] branch aip-11 updated (4ab2599 -> 6973513)

2019-11-19 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a change to branch aip-11 in repository https://gitbox.apache.org/repos/asf/airflow-site.git. from 4ab2599 Add meta description to head (#185) add 6973513 Add archtetypes (#184) No new revisions w

[GitHub] [airflow-site] mik-laj merged pull request #184: Add archtetypes

2019-11-19 Thread GitBox
mik-laj merged pull request #184: Add archtetypes URL: https://github.com/apache/airflow-site/pull/184 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [airflow] codecov-io edited a comment on issue #6461: [AIRFLOW-5804] Batch the xcom pull operation

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6461: [AIRFLOW-5804] Batch the xcom pull operation URL: https://github.com/apache/airflow/pull/6461#issuecomment-552133386 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6461?src=pr&el=h1) Report > Merging [#6461](https://codecov.io/gh/apach

[jira] [Resolved] (AIRFLOW-6001) Lazy load CLI commands

2019-11-19 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula resolved AIRFLOW-6001. Fix Version/s: 1.10.7 Resolution: Fixed > Lazy load CLI commands > ---

[jira] [Created] (AIRFLOW-6017) Exclude PULL_REQUEST_TEMPLATE.md from RAT check

2019-11-19 Thread Jarek Potiuk (Jira)
Jarek Potiuk created AIRFLOW-6017: - Summary: Exclude PULL_REQUEST_TEMPLATE.md from RAT check Key: AIRFLOW-6017 URL: https://issues.apache.org/jira/browse/AIRFLOW-6017 Project: Apache Airflow

[GitHub] [airflow] codecov-io edited a comment on issue #6447: [AIRFLOW-5792] Straighten out the migrations

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6447: [AIRFLOW-5792] Straighten out the migrations URL: https://github.com/apache/airflow/pull/6447#issuecomment-546722032 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6447?src=pr&el=h1) Report > Merging [#6447](https://codecov.io/gh/apach

[GitHub] [airflow] potiuk opened a new pull request #6611: [AIRFLOW-6017] Exclude PULL_REQUEST_TEMPLATE.md from RAT check

2019-11-19 Thread GitBox
potiuk opened a new pull request #6611: [AIRFLOW-6017] Exclude PULL_REQUEST_TEMPLATE.md from RAT check URL: https://github.com/apache/airflow/pull/6611 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apa

[jira] [Commented] (AIRFLOW-6017) Exclude PULL_REQUEST_TEMPLATE.md from RAT check

2019-11-19 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6017?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16977898#comment-16977898 ] ASF GitHub Bot commented on AIRFLOW-6017: - potiuk commented on pull request #66

[GitHub] [airflow] codecov-io edited a comment on issue #6447: [AIRFLOW-5792] Straighten out the migrations

2019-11-19 Thread GitBox
codecov-io edited a comment on issue #6447: [AIRFLOW-5792] Straighten out the migrations URL: https://github.com/apache/airflow/pull/6447#issuecomment-546722032 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6447?src=pr&el=h1) Report > Merging [#6447](https://codecov.io/gh/apach

[jira] [Resolved] (AIRFLOW-5893) Refactor CLI tests

2019-11-19 Thread Kamil Bregula (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5893?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kamil Bregula resolved AIRFLOW-5893. Fix Version/s: 1.10.7 Resolution: Fixed > Refactor CLI tests > --

  1   2   >