[GitHub] [airflow] codecov-io edited a comment on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#issuecomment-569523812 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6950?src=pr=h1) Report > Merging

[GitHub] [airflow] mik-laj commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
mik-laj commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569718112 @ashb 1. https://github.com/apache/airflow/blob/master/airflow/cli/commands/task_command.py#L142 2.

[GitHub] [airflow-site] potiuk commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362024088 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0 +1,43

[GitHub] [airflow] ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569717105 > @ashb I tested locally and this does not happen because a new Dagbag is created each time CLI command is called. Wait, do

[GitHub] [airflow-site] potiuk commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362020195 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0 +1,43

[GitHub] [airflow] ashb commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
ashb commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#discussion_r362019838 ## File path: airflow/executors/celery_executor.py ## @@ -63,13 +62,9 @@ def

[GitHub] [airflow] ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569713841 > @ashb I tested locally and this does not happen because a new Dagbag is created each time CLI command is called. Oh yes,

[GitHub] [airflow] kaxil commented on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#issuecomment-569713783 > Updated all comments @kaxil ! What shall we do with the backwards compatibility there? Do you think we should worry

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362018719 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362018719 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[jira] [Resolved] (AIRFLOW-6109) Rename function service

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk resolved AIRFLOW-6109. --- Fix Version/s: 2.0.0 Resolution: Fixed > Rename function service >

[GitHub] [airflow] potiuk commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x)

2019-12-30 Thread GitBox
potiuk commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x) URL: https://github.com/apache/airflow/pull/6930#issuecomment-569713087  This is an automated message from the Apache Git

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362018560 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362018302 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[airflow-site] branch master updated: Add Elena's GSOD impressions (#225)

2019-12-30 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow-site.git The following commit(s) were added to refs/heads/master by this push: new 41eb4bb Add Elena's GSOD impressions

[GitHub] [airflow-site] potiuk merged pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
potiuk merged pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow-site] potiuk commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r362017522 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0 +1,43

[jira] [Commented] (AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17005398#comment-17005398 ] ASF GitHub Bot commented on AIRFLOW-6402: - ohadmata commented on pull request #6970:

[GitHub] [airflow] potiuk commented on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on issue #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#issuecomment-569711638 Updated all comments @kaxil ! What shall we do with the backwards compatibility there? Do you think we should worry

[GitHub] [airflow] ohadmata opened a new pull request #6970: [AIRFLOW-6402] SSH operator exception - NoneType object has no attribute startswith

2019-12-30 Thread GitBox
ohadmata opened a new pull request #6970: [AIRFLOW-6402] SSH operator exception - NoneType object has no attribute startswith URL: https://github.com/apache/airflow/pull/6970 fix exception This is an automated

[jira] [Updated] (AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread ohad (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6402?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ohad updated AIRFLOW-6402: -- Summary: SSH operator exception - 'NoneType' object has no attribute 'startswith' (was: SSH operator

[jira] [Updated] (AIRFLOW-6406) Operator Resources have wronge type hint derived

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6406?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-6406: -- Summary: Operator Resources have wronge type hint derived (was: Operator Resources miss the

[jira] [Commented] (AIRFLOW-6402) SSH operator exceotion - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17005393#comment-17005393 ] ASF GitHub Bot commented on AIRFLOW-6402: - ohadmata commented on pull request #6969:

[GitHub] [airflow] ohadmata opened a new pull request #6969: [\[AIRFLOW-6402\]](https://issues.apache.org/jira/browse/AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startsw

2019-12-30 Thread GitBox
ohadmata opened a new pull request #6969: [\[AIRFLOW-6402\]](https://issues.apache.org/jira/browse/AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startswith' URL: https://github.com/apache/airflow/pull/6969 fix ssh operator constructor

[GitHub] [airflow] ohadmata closed pull request #6969: [\[AIRFLOW-6402\]](https://issues.apache.org/jira/browse/AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread GitBox
ohadmata closed pull request #6969: [\[AIRFLOW-6402\]](https://issues.apache.org/jira/browse/AIRFLOW-6402) SSH operator exception - 'NoneType' object has no attribute 'startswith' URL: https://github.com/apache/airflow/pull/6969

[jira] [Commented] (AIRFLOW-6402) SSH operator exceotion - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17005391#comment-17005391 ] ASF GitHub Bot commented on AIRFLOW-6402: - ohadmata commented on pull request #6969:

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362014321 ## File path: airflow/utils/helpers.py ## @@ -142,83 +135,6 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362014068 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362014133 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362013565 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[jira] [Assigned] (AIRFLOW-6402) SSH operator exceotion - 'NoneType' object has no attribute 'startswith'

2019-12-30 Thread ohad (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6402?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ohad reassigned AIRFLOW-6402: - Assignee: ohad > SSH operator exceotion - 'NoneType' object has no attribute 'startswith' >

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362013100 ## File path: airflow/models/baseoperator.py ## @@ -670,6 +672,7 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362011152 ## File path: airflow/models/baseoperator.py ## @@ -862,6 +865,7 @@ def clear(self,

[jira] [Created] (AIRFLOW-6406) Operator Resources miss the type hint derived

2019-12-30 Thread Jarek Potiuk (Jira)
Jarek Potiuk created AIRFLOW-6406: - Summary: Operator Resources miss the type hint derived Key: AIRFLOW-6406 URL: https://issues.apache.org/jira/browse/AIRFLOW-6406 Project: Apache Airflow

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362010644 ## File path: tests/models/test_baseoperator.py ## @@ -241,7 +243,46 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362009498 ## File path: tests/models/test_baseoperator.py ## @@ -241,7 +243,46 @@ def

[GitHub] [airflow] kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#discussion_r362009139 ## File path: airflow/executors/celery_executor.py ## @@ -63,13 +62,9 @@ def

[GitHub] [airflow] michalslowikowski00 opened a new pull request #6968: [Airflow-6110] [AIP-21] Rename GCP natural language operators

2019-12-30 Thread GitBox
michalslowikowski00 opened a new pull request #6968: [Airflow-6110] [AIP-21] Rename GCP natural language operators URL: https://github.com/apache/airflow/pull/6968 PR contains changes regarding GCP natural language operators: - renamed classes: CloudLanguageAnalyzeEntitiesOperator,

[GitHub] [airflow] kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#discussion_r362008246 ## File path: airflow/executors/celery_executor.py ## @@ -63,13 +62,9 @@ def

[GitHub] [airflow] potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r362007720 ## File path: airflow/utils/helpers.py ## @@ -28,15 +28,8 @@ import psutil from

[GitHub] [airflow] kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#discussion_r362007109 ## File path: airflow/executors/celery_executor.py ## @@ -63,13 +62,9 @@ def

[GitHub] [airflow] kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#discussion_r362006526 ## File path: airflow/bin/cli.py ## @@ -1063,3 +1063,26 @@ def _add_subcommand(cls,

[GitHub] [airflow] phstudy commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x)

2019-12-30 Thread GitBox
phstudy commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x) URL: https://github.com/apache/airflow/pull/6930#issuecomment-569697351 @potiuk I found you said > Usually only committers cherry-pick changes to v1-10 in #airflow-how-to-pr

[jira] [Created] (AIRFLOW-6405) Bigquery Update Table Properties Operator

2019-12-30 Thread Jithin Sukumar (Jira)
Jithin Sukumar created AIRFLOW-6405: --- Summary: Bigquery Update Table Properties Operator Key: AIRFLOW-6405 URL: https://issues.apache.org/jira/browse/AIRFLOW-6405 Project: Apache Airflow

[GitHub] [airflow] codecov-io edited a comment on issue #6966: [AIRFLOW-6108] [AIP-21] Rename CloudDLP operators

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6966: [AIRFLOW-6108] [AIP-21] Rename CloudDLP operators URL: https://github.com/apache/airflow/pull/6966#issuecomment-569666098 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6966?src=pr=h1) Report > Merging

[jira] [Commented] (AIRFLOW-6403) BigQuery - add tests for BigQueryCursor

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

[GitHub] [airflow] potiuk merged pull request #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
potiuk merged pull request #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967 This is an automated message from the Apache Git Service. To respond to the message,

[jira] [Resolved] (AIRFLOW-6403) BigQuery - add tests for BigQueryCursor

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6403?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk resolved AIRFLOW-6403. --- Fix Version/s: 2.0.0 Resolution: Fixed > BigQuery - add tests for BigQueryCursor >

[jira] [Commented] (AIRFLOW-6403) BigQuery - add tests for BigQueryCursor

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

[jira] [Commented] (AIRFLOW-6394) Simplify github template

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

[GitHub] [airflow] mik-laj commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
mik-laj commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569692530 @ashb I tested locally and this does not happen because a new Dagbag is created each time CLI command is called. I would also

[GitHub] [airflow] potiuk edited a comment on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x)

2019-12-30 Thread GitBox
potiuk edited a comment on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x) URL: https://github.com/apache/airflow/pull/6930#issuecomment-569692518 @phstudy -> Why did you close that one ? Would you still want to have this change to be cherry-picked to 1.10 when

[GitHub] [airflow] potiuk commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x)

2019-12-30 Thread GitBox
potiuk commented on issue #6930: [AIRFLOW-6372] Align WASB remote logging URI scheme (1.10.x) URL: https://github.com/apache/airflow/pull/6930#issuecomment-569692518 @phstudy -> Why did you close that one ? Would you want to have this change to be cherry-picked to 1.10 when it's merged?

[GitHub] [airflow] potiuk commented on issue #6963: [AIRFLOW-XXX] fix gcp keyfile_dict typo (1.10.x)

2019-12-30 Thread GitBox
potiuk commented on issue #6963: [AIRFLOW-XXX] fix gcp keyfile_dict typo (1.10.x) URL: https://github.com/apache/airflow/pull/6963#issuecomment-569691794 Yep. I cherry-picked it to 1.10 :). This is an automated message from

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r361996822 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996564 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996665 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow-site] mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #225: Add Elena's GSOD impressions URL: https://github.com/apache/airflow-site/pull/225#discussion_r361996572 ## File path: landing-pages/site/content/en/blog/documenting-using-local-development-environments.md ## @@ -0,0

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996372 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996350 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996372 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] ashb edited a comment on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
ashb edited a comment on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569688311 Broadly sounds good, however as it stands this PR would load the dag in to the long running Celery worker process. Which,

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996196 ## File path: airflow/models/baseoperator.py ## @@ -670,6 +672,7 @@ def

[GitHub] [airflow] ashb commented on issue #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram

2019-12-30 Thread GitBox
ashb commented on issue #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram URL: https://github.com/apache/airflow/pull/6961#issuecomment-569689176 I'm not sure your diagram is right around the "Assign tasks to worker process" part - my understanding was that the

[GitHub] [airflow] potiuk commented on a change in pull request #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram

2019-12-30 Thread GitBox
potiuk commented on a change in pull request #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram URL: https://github.com/apache/airflow/pull/6961#discussion_r361995737 ## File path: docs/img/run_task_on_celery_executor.puml ## @@ -0,0 +1,77 @@ +/**

[jira] [Commented] (AIRFLOW-4445) [security] mushroom cloud errors too verbose

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

[jira] [Commented] (AIRFLOW-6057) Add op_args and op_kwargs in PythonSensor templated fields

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

[jira] [Commented] (AIRFLOW-6397) Check for `sub_process` before trying to get pid in bash operator on kill

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

[jira] [Commented] (AIRFLOW-4445) [security] mushroom cloud errors too verbose

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

[jira] [Commented] (AIRFLOW-6400) pytest not working on Windows

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

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361995214 ## File path: tests/models/test_baseoperator.py ## @@ -241,7 +243,46 @@ def

[jira] [Commented] (AIRFLOW-6352) security - ui - add timeout

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

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361995400 ## File path: airflow/models/baseoperator.py ## @@ -862,6 +865,7 @@ def clear(self,

[jira] [Commented] (AIRFLOW-5149) Config flag to skip SLA checks

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

[jira] [Commented] (AIRFLOW-5149) Config flag to skip SLA checks

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

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361995054 ## File path: airflow/utils/helpers.py ## @@ -142,83 +135,6 @@ def

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task

2019-12-30 Thread GitBox
ashb commented on issue #6905: [AIRFLOW-6361] Run LocalTaskJob directly in Celery task URL: https://github.com/apache/airflow/pull/6905#issuecomment-569688311 Broadly sounds good, however as it stands this PR would load the dag in to the long running Celery worker process.

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361994621 ## File path: airflow/utils/helpers.py ## @@ -28,15 +28,8 @@ import psutil from

[GitHub] [airflow] mik-laj commented on issue #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram

2019-12-30 Thread GitBox
mik-laj commented on issue #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram URL: https://github.com/apache/airflow/pull/6961#issuecomment-569687921 @potiuk I added license and pre-commit check.

[GitHub] [airflow] mik-laj commented on a change in pull request #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram

2019-12-30 Thread GitBox
mik-laj commented on a change in pull request #6961: [AIRFLOW-XXX] Add task execution process on Celery Execution diagram URL: https://github.com/apache/airflow/pull/6961#discussion_r361994309 ## File path: docs/img/run_task_on_celery_executor.puml ## @@ -0,0 +1,55 @@

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361993905 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[jira] [Updated] (AIRFLOW-6397) Check for `sub_process` before trying to get pid in bash operator on kill

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6397?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-6397: -- Fix Version/s: (was: 2.0.0) 1.10.8 > Check for `sub_process` before

[GitHub] [airflow] kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361993905 ## File path: UPDATING.md ## @@ -57,6 +57,31 @@

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[jira] [Resolved] (AIRFLOW-6400) pytest not working on Windows

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6400?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk resolved AIRFLOW-6400. --- Fix Version/s: 1.10.8 Resolution: Fixed > pytest not working on Windows >

[GitHub] [airflow] codecov-io commented on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io commented on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[GitHub] [airflow] codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor

2019-12-30 Thread GitBox
codecov-io edited a comment on issue #6967: [AIRFLOW-6403] BigQuery - add tests for BigQueryCursor URL: https://github.com/apache/airflow/pull/6967#issuecomment-569687060 # [Codecov](https://codecov.io/gh/apache/airflow/pull/6967?src=pr=h1) Report > Merging

[jira] [Commented] (AIRFLOW-6394) Simplify github template

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

[jira] [Commented] (AIRFLOW-6394) Simplify github template

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

[jira] [Resolved] (AIRFLOW-6394) Simplify github template

2019-12-30 Thread Kaxil Naik (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-6394?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Kaxil Naik resolved AIRFLOW-6394. - Fix Version/s: 1.10.8 2.0.0 Resolution: Fixed > Simplify github

[GitHub] [airflow] kaxil merged pull request #6955: [AIRFLOW-6394] Simplify github PR template

2019-12-30 Thread GitBox
kaxil merged pull request #6955: [AIRFLOW-6394] Simplify github PR template URL: https://github.com/apache/airflow/pull/6955 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] kaxil commented on a change in pull request #6955: [AIRFLOW-6394] Simplify github PR template

2019-12-30 Thread GitBox
kaxil commented on a change in pull request #6955: [AIRFLOW-6394] Simplify github PR template URL: https://github.com/apache/airflow/pull/6955#discussion_r361992084 ## File path: .github/PULL_REQUEST_TEMPLATE.md ## @@ -1,33 +1,9 @@ -Make sure you have checked _all_ steps

[jira] [Updated] (AIRFLOW-4445) [security] mushroom cloud errors too verbose

2019-12-30 Thread Jarek Potiuk (Jira)
[ https://issues.apache.org/jira/browse/AIRFLOW-4445?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk updated AIRFLOW-4445: -- Fix Version/s: (was: 2.0.0) 1.10.8 > [security] mushroom cloud errors

<    1   2   3   >