[GitHub] [airflow] OmerJog commented on issue #6261: [AIRFLOW-5141] Added optional resource usage logging to KubernetesPod…

2019-12-02 Thread GitBox
OmerJog commented on issue #6261: [AIRFLOW-5141] Added optional resource usage logging to KubernetesPod… URL: https://github.com/apache/airflow/pull/6261#issuecomment-561041583 @Sinsin1367 there is a conflict to resolve This

[jira] [Commented] (AIRFLOW-6097) AIP-21: Rename services

2019-12-02 Thread Jira
[ https://issues.apache.org/jira/browse/AIRFLOW-6097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986678#comment-16986678 ] Michał Słowikowski commented on AIRFLOW-6097: - [~kaxilnaik] Yes, I have a list of modules

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

2019-12-02 Thread afusr (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6014?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986672#comment-16986672 ] afusr commented on AIRFLOW-6014: [~dimberman] Do you think this can be merged? This fixed has saved us

[jira] [Created] (AIRFLOW-6161) GoogleCloudBaseHook delegate_to doesn't work with GCE service account

2019-12-02 Thread Guillaume Poulin (Jira)
Guillaume Poulin created AIRFLOW-6161: - Summary: GoogleCloudBaseHook delegate_to doesn't work with GCE service account Key: AIRFLOW-6161 URL: https://issues.apache.org/jira/browse/AIRFLOW-6161

[jira] [Resolved] (AIRFLOW-6069) Host Travis build should use always python 3.6

2019-12-02 Thread Kaxil Naik (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6069?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-6069. - Resolution: Fixed > Host Travis build should use always python 3.6 >

[jira] [Closed] (AIRFLOW-6097) AIP-21: Rename services

2019-12-02 Thread Kaxil Naik (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6097?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik closed AIRFLOW-6097. --- Resolution: Implemented Some of these are already implemented /changed. Have you checked the list

[jira] [Resolved] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread Kaxil Naik (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-6159. - Fix Version/s: 1.10.7 Resolution: Fixed > Change logging level of the heartbeat message

[jira] [Commented] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986525#comment-16986525 ] ASF subversion and git services commented on AIRFLOW-6159: -- Commit

[jira] [Commented] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986524#comment-16986524 ] ASF GitHub Bot commented on AIRFLOW-6159: - kaxil commented on pull request #6716:

[GitHub] [airflow] kaxil merged pull request #6716: [AIRFLOW-6159] Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread GitBox
kaxil merged pull request #6716: [AIRFLOW-6159] Change logging level of the heartbeat message to DEBUG URL: https://github.com/apache/airflow/pull/6716 This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] ITriangle closed pull request #6707: Fix task get log

2019-12-02 Thread GitBox
ITriangle closed pull request #6707: Fix task get log URL: https://github.com/apache/airflow/pull/6707 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[jira] [Commented] (AIRFLOW-5540) task with SparkSubmitOperator does not fail if the spark job it executes fails.

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986511#comment-16986511 ] ASF GitHub Bot commented on AIRFLOW-5540: - stale[bot] commented on pull request #6288:

[GitHub] [airflow] stale[bot] closed pull request #6288: AIRFLOW-5540 adding rise exception when the _resolve_connection method fails

2019-12-02 Thread GitBox
stale[bot] closed pull request #6288: AIRFLOW-5540 adding rise exception when the _resolve_connection method fails URL: https://github.com/apache/airflow/pull/6288 This is an automated message from the Apache Git Service.

[GitHub] [airflow] codecov-io commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
codecov-io commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#issuecomment-560962980 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6715?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#issuecomment-560962980 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6715?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6599#issuecomment-560146249 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6599?src=pr=h1) Report >

[GitHub] [airflow] codecov-io edited a comment on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6599: [AIRFLOW-6007] Check providers instead of provider package. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6599#issuecomment-560146249 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6599?src=pr=h1) Report >

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352943542 ## File path: airflow/utils/dag_processing.py ## @@ -550,8 +549,8 @@ def

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352943496 ## File path: airflow/logging_config.py ## @@ -1,3 +1,4 @@ +# -*- coding: utf-8 -*-

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352942991 ## File path: airflow/executors/kubernetes_executor.py ## @@ -256,15 +256,15 @@ class

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352942743 ## File path: airflow/executors/dask_executor.py ## @@ -66,20 +63,17 @@ def

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352942604 ## File path: airflow/__init__.py ## @@ -44,23 +44,8 @@ settings.initialize()

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352941360 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352941360 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

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

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352941360 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

[jira] [Commented] (AIRFLOW-6160) Move _check_and_change_state_before_execution to LocalTaskJob

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986485#comment-16986485 ] ASF GitHub Bot commented on AIRFLOW-6160: - mik-laj commented on pull request #6717:

[GitHub] [airflow] mik-laj opened a new pull request #6717: [AIRFLOW-6160] Move _check_and_change_state_before_execution to LocalTaskJob

2019-12-02 Thread GitBox
mik-laj opened a new pull request #6717: [AIRFLOW-6160] Move _check_and_change_state_before_execution to LocalTaskJob URL: https://github.com/apache/airflow/pull/6717 Hello I think that LocaclTaskJob should control which tasks are to be executed. SchedulerJob and BackfillingJob

[GitHub] [airflow] codecov-io edited a comment on issue #6696: [AIRFLOW-6128] Simplify plugins_manager and webserver plugin code. Based on [AIRFLOW-6004] [AIRFLOW-6140]

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6696: [AIRFLOW-6128] Simplify plugins_manager and webserver plugin code. Based on [AIRFLOW-6004] [AIRFLOW-6140] URL: https://github.com/apache/airflow/pull/6696#issuecomment-560183548 #

[jira] [Created] (AIRFLOW-6160) Move _check_and_change_state_before_execution to LocalTaskJob

2019-12-02 Thread Kamil Bregula (Jira)
Kamil Bregula created AIRFLOW-6160: -- Summary: Move _check_and_change_state_before_execution to LocalTaskJob Key: AIRFLOW-6160 URL: https://issues.apache.org/jira/browse/AIRFLOW-6160 Project: Apache

[GitHub] [airflow] codecov-io edited a comment on issue #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6702#issuecomment-560257785 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6702?src=pr=h1) Report > Merging

[GitHub] [airflow] kaxil commented on a change in pull request #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6702#discussion_r352922159 ## File path: airflow/models/baseoperator.py ## @@ -869,11 +877,16

[GitHub] [airflow] codecov-io edited a comment on issue #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6702#issuecomment-560257785 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6702?src=pr=h1) Report > Merging

[GitHub] [airflow] kaxil commented on a change in pull request #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004]

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6702: [AIRFLOW-6140] Add missing types for some core classes. Depends on [AIRFLOW-6004] URL: https://github.com/apache/airflow/pull/6702#discussion_r352921900 ## File path: airflow/executors/executor_loader.py ## @@ -25,11

[GitHub] [airflow] mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352892463 ## File path: airflow/jobs/scheduler_job.py ## @@ -54,8 +54,9 @@ from

[GitHub] [airflow] kaxil commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
kaxil commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#issuecomment-560911546 > @kaxil I agree, but for now I am trying to solve one problem. This change is already very big - +2,256

[GitHub] [airflow] mik-laj commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#issuecomment-560909396 @Kaxil I agree, but for now I am trying to solve one problem. This change is already very big - +2,256

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352916680 ## File path: airflow/logging_config.py ## @@ -1,3 +1,4 @@ +# -*- coding: utf-8 -*-

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352916873 ## File path: airflow/utils/dag_processing.py ## @@ -550,8 +549,8 @@ def

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352916555 ## File path: airflow/executors/kubernetes_executor.py ## @@ -256,15 +256,15 @@ class

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352916409 ## File path: airflow/executors/dask_executor.py ## @@ -66,20 +63,17 @@ def

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352915970 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352915706 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

[GitHub] [airflow] codecov-io edited a comment on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714#issuecomment-560887984 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6714?src=pr=h1) Report > Merging

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352914727 ## File path: airflow/executors/dask_executor.py ## @@ -36,9 +35,7 @@ def

[GitHub] [airflow] codecov-io commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
codecov-io commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714#issuecomment-560887984 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6714?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports

2019-12-02 Thread GitBox
codecov-io edited a comment on issue #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#issuecomment-559224555 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6596?src=pr=h1) Report > Merging

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

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#discussion_r352911855 ## File path: airflow/__init__.py ## @@ -44,23 +44,8 @@ settings.initialize()

[GitHub] [airflow] kaxil commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
kaxil commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#issuecomment-560857645 Just read Ash's comment and I think it might be good to remove Abstract* versions ! >I think that the

[GitHub] [airflow] kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352907259 ## File path: airflow/jobs/scheduler_job.py ## @@ -290,105 +292,28 @@ def

[GitHub] [airflow] kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352907442 ## File path: airflow/jobs/scheduler_job.py ## @@ -290,105 +292,28 @@ def

[GitHub] [airflow] kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
kaxil commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352906748 ## File path: airflow/jobs/scheduler_job.py ## @@ -54,8 +54,9 @@ from

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352903155 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352903155 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352908365 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352908491 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352908300 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] XD-DENG merged pull request #6708: [AIRFLOW-XXX] Fix docstring minor issues in airflow/kubernetes/

2019-12-02 Thread GitBox
XD-DENG merged pull request #6708: [AIRFLOW-XXX] Fix docstring minor issues in airflow/kubernetes/ URL: https://github.com/apache/airflow/pull/6708 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] kaxil opened a new pull request #6716: [AIRFLOW-6159] Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread GitBox
kaxil opened a new pull request #6716: [AIRFLOW-6159] Change logging level of the heartbeat message to DEBUG URL: https://github.com/apache/airflow/pull/6716 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986444#comment-16986444 ] ASF GitHub Bot commented on AIRFLOW-6159: - kaxil commented on pull request #6716:

[jira] [Updated] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread Kaxil Naik (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik updated AIRFLOW-6159: Description: As mentioned in

[jira] [Created] (AIRFLOW-6159) Change logging level of the heartbeat message to DEBUG

2019-12-02 Thread Kaxil Naik (Jira)
Kaxil Naik created AIRFLOW-6159: --- Summary: Change logging level of the heartbeat message to DEBUG Key: AIRFLOW-6159 URL: https://issues.apache.org/jira/browse/AIRFLOW-6159 Project: Apache Airflow

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352903466 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352903155 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[jira] [Created] (AIRFLOW-6158) Upgrade sendgrid dependency

2019-12-02 Thread Marcin Szymanski (Jira)
Marcin Szymanski created AIRFLOW-6158: - Summary: Upgrade sendgrid dependency Key: AIRFLOW-6158 URL: https://issues.apache.org/jira/browse/AIRFLOW-6158 Project: Apache Airflow Issue Type:

[GitHub] [airflow] tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook

2019-12-02 Thread GitBox
tfindlay-au commented on a change in pull request #6604: [AIRFLOW-5920] Neo4j operator and hook URL: https://github.com/apache/airflow/pull/6604#discussion_r352901973 ## File path: airflow/contrib/hooks/neo4j_hook.py ## @@ -0,0 +1,101 @@ +# -*- coding: utf-8 -*- +# +#

[GitHub] [airflow] kaxil commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
kaxil commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#issuecomment-560795290 > What do you think to save the generated links in a separate column in the database after Task execution? I

[GitHub] [airflow] mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352895446 ## File path: airflow/jobs/scheduler_job.py ## @@ -54,8 +54,9 @@ from

[GitHub] [airflow] mik-laj commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on issue #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#issuecomment-560761473 I would love to wait for Jarek's changes. I am pleased with the warm words about this change.

[GitHub] [airflow] mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352893158 ## File path: airflow/jobs/__init__.py ## @@ -23,4 +23,4 @@ from

[GitHub] [airflow] mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class

2019-12-02 Thread GitBox
mik-laj commented on a change in pull request #6697: [AIRFLOW-6135] Extract DAG processing from SchedulerJob into separate class URL: https://github.com/apache/airflow/pull/6697#discussion_r352892463 ## File path: airflow/jobs/scheduler_job.py ## @@ -54,8 +54,9 @@ from

[GitHub] [airflow] mik-laj commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
mik-laj commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#issuecomment-560746534 What do you think to save the generated links in a separate column in the database after Task execution? I

[GitHub] [airflow] kaxil edited a comment on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
kaxil edited a comment on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714#issuecomment-560726423 > @potiuk We once talked that DAG serialization can be done by serializing class arguments in the database. This

[GitHub] [airflow] kaxil commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
kaxil commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714#issuecomment-560726423 > @potiuk We once talked that DAG serialization can be done by serializing class arguments in the database. This PR

[GitHub] [airflow] mik-laj commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
mik-laj commented on issue #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714#issuecomment-560715143 @potiuk We once talked that DAG serialization can be done by serializing class arguments in the database. This PR

[jira] [Commented] (AIRFLOW-5945) Fix limitation with OperatorLinks when using DAG Serialization

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-5945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986419#comment-16986419 ] ASF GitHub Bot commented on AIRFLOW-5945: - kaxil commented on pull request #6715:

[GitHub] [airflow] kaxil commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
kaxil commented on issue #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715#issuecomment-560711709 This PR is dependent on #6714 so please only review

[GitHub] [airflow] kaxil opened a new pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization

2019-12-02 Thread GitBox
kaxil opened a new pull request #6715: [AIRFLOW-5945] Make inbuilt OperatorLinks work when using Serialization URL: https://github.com/apache/airflow/pull/6715 Follow Up PR of https://github.com/apache/airflow/pull/6714 Make sure you have checked _all_ steps below. ### Jira

[jira] [Created] (AIRFLOW-6157) Run multiple executors

2019-12-02 Thread Jarek Potiuk (Jira)
Jarek Potiuk created AIRFLOW-6157: - Summary: Run multiple executors Key: AIRFLOW-6157 URL: https://issues.apache.org/jira/browse/AIRFLOW-6157 Project: Apache Airflow Issue Type: Improvement

[GitHub] [airflow] dougblack closed pull request #6673: Pod aliases

2019-12-02 Thread GitBox
dougblack closed pull request #6673: Pod aliases URL: https://github.com/apache/airflow/pull/6673 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[jira] [Commented] (AIRFLOW-6153) Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986399#comment-16986399 ] ASF GitHub Bot commented on AIRFLOW-6153: - kaxil commented on pull request #6714:

[GitHub] [airflow] kaxil opened a new pull request #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators

2019-12-02 Thread GitBox
kaxil opened a new pull request #6714: [AIRFLOW-6153] Allow adding Custom Fields to Serialized Operators URL: https://github.com/apache/airflow/pull/6714 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[GitHub] [airflow] codecov-io commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor

2019-12-02 Thread GitBox
codecov-io commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor URL: https://github.com/apache/airflow/pull/6683#issuecomment-560622552 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6683?src=pr=h1) Report > Merging

[GitHub] [airflow] jomach commented on issue #6694: AIRFLOW-6132 - Allow to pass in tags

2019-12-02 Thread GitBox
jomach commented on issue #6694: AIRFLOW-6132 - Allow to pass in tags URL: https://github.com/apache/airflow/pull/6694#issuecomment-560612984 @potiuk can we merge ? This is an automated message from the Apache Git Service.

[GitHub] [airflow] potiuk commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor

2019-12-02 Thread GitBox
potiuk commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor URL: https://github.com/apache/airflow/pull/6683#issuecomment-560583506 I restarted the failing job. It's a flaky "run_on_kill" test that @nuclearpinguin is solving with the upcoming #6472

[GitHub] [airflow] zuku1985 commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor

2019-12-02 Thread GitBox
zuku1985 commented on issue #6683: [AIRFLOW-6019] Add flushing in execute method for BigQueryCursor URL: https://github.com/apache/airflow/pull/6683#issuecomment-560581631 @nuclearpinguin any hints what should I do to fix failing tests? I ran out of ideas :/

[jira] [Created] (AIRFLOW-6156) sla_miss_callback on baseoperator for task level

2019-12-02 Thread Chengzhi Zhao (Jira)
Chengzhi Zhao created AIRFLOW-6156: -- Summary: sla_miss_callback on baseoperator for task level Key: AIRFLOW-6156 URL: https://issues.apache.org/jira/browse/AIRFLOW-6156 Project: Apache Airflow

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6713: Add some r/w functionality to GCP Bigtable Hook

2019-12-02 Thread GitBox
nuclearpinguin commented on a change in pull request #6713: Add some r/w functionality to GCP Bigtable Hook URL: https://github.com/apache/airflow/pull/6713#discussion_r352818753 ## File path: airflow/gcp/hooks/bigtable.py ## @@ -276,3 +278,58 @@ def

[GitHub] [airflow] nuclearpinguin commented on a change in pull request #6713: Add some r/w functionality to GCP Bigtable Hook

2019-12-02 Thread GitBox
nuclearpinguin commented on a change in pull request #6713: Add some r/w functionality to GCP Bigtable Hook URL: https://github.com/apache/airflow/pull/6713#discussion_r352819063 ## File path: airflow/gcp/hooks/bigtable.py ## @@ -276,3 +278,58 @@ def

[jira] [Commented] (AIRFLOW-6155) Add some reading and writing functionality to the GCP Bigtable hook

2019-12-02 Thread Emily Mazo (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986296#comment-16986296 ] Emily Mazo commented on AIRFLOW-6155: - PR: [https://github.com/apache/airflow/pull/6713] > Add

[GitHub] [airflow] EmilyMazo opened a new pull request #6713: Add some r/w functionality to GCP Bigtable Hook

2019-12-02 Thread GitBox
EmilyMazo opened a new pull request #6713: Add some r/w functionality to GCP Bigtable Hook URL: https://github.com/apache/airflow/pull/6713 Make sure you have checked _all_ steps below. ### Jira - [ X] My PR addresses the following [Airflow

[jira] [Created] (AIRFLOW-6155) Add some reading and writing functionality to the GCP Bigtable hook

2019-12-02 Thread Emily Mazo (Jira)
Emily Mazo created AIRFLOW-6155: --- Summary: Add some reading and writing functionality to the GCP Bigtable hook Key: AIRFLOW-6155 URL: https://issues.apache.org/jira/browse/AIRFLOW-6155 Project: Apache

[GitHub] [airflow] codecov-io commented on issue #6712: [AIRFLOW-XXX] add notice for Mesos Executor deprecation in docs

2019-12-02 Thread GitBox
codecov-io commented on issue #6712: [AIRFLOW-XXX] add notice for Mesos Executor deprecation in docs URL: https://github.com/apache/airflow/pull/6712#issuecomment-560535226 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6712?src=pr=h1) Report > Merging

[GitHub] [airflow] potiuk commented on issue #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports

2019-12-02 Thread GitBox
potiuk commented on issue #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#issuecomment-560527624 I think NOW I resolved everything that needs to be resolved in this commit (some more commits are following), I let

[GitHub] [airflow] potiuk commented on a change in pull request #6585: More GSOD improvements

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6585: More GSOD improvements URL: https://github.com/apache/airflow/pull/6585#discussion_r352763339 ## File path: CONTRIBUTING.rst ## @@ -742,8 +495,56 @@ Resources & links - Airflow users mailing list: ``_ -- `Issues on

[GitHub] [airflow] potiuk commented on a change in pull request #6585: More GSOD improvements

2019-12-02 Thread GitBox
potiuk commented on a change in pull request #6585: More GSOD improvements URL: https://github.com/apache/airflow/pull/6585#discussion_r352762271 ## File path: CONTRIBUTING.rst ## @@ -742,8 +495,56 @@ Resources & links - Airflow users mailing list: ``_ -- `Issues on

[jira] [Commented] (AIRFLOW-6142) Different results for pylint locally and in Travis CI

2019-12-02 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986254#comment-16986254 ] ASF GitHub Bot commented on AIRFLOW-6142: - potiuk commented on pull request #6705:

[jira] [Commented] (AIRFLOW-6142) Different results for pylint locally and in Travis CI

2019-12-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986255#comment-16986255 ] ASF subversion and git services commented on AIRFLOW-6142: -- Commit

[jira] [Resolved] (AIRFLOW-6142) Different results for pylint locally and in Travis CI

2019-12-02 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6142?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk resolved AIRFLOW-6142. --- Fix Version/s: 2.0.0 Resolution: Fixed > Different results for pylint locally and

[jira] [Commented] (AIRFLOW-6142) Different results for pylint locally and in Travis CI

2019-12-02 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6142?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16986256#comment-16986256 ] ASF subversion and git services commented on AIRFLOW-6142: -- Commit

[GitHub] [airflow] potiuk merged pull request #6705: [AIRFLOW-6142] Fix different local/Travis pylint results

2019-12-02 Thread GitBox
potiuk merged pull request #6705: [AIRFLOW-6142] Fix different local/Travis pylint results URL: https://github.com/apache/airflow/pull/6705 This is an automated message from the Apache Git Service. To respond to the

  1   2   3   >