[GitHub] [airflow] jhtimmins opened a new pull request #10540: Fix typo. Sucess to Success.

2020-08-24 Thread GitBox
jhtimmins opened a new pull request #10540: URL: https://github.com/apache/airflow/pull/10540 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for

[GitHub] [airflow] syudb commented on pull request #10461: Updated REST API call so GET requests pass payload in query string instead of request body

2020-08-24 Thread GitBox
syudb commented on pull request #10461: URL: https://github.com/apache/airflow/pull/10461#issuecomment-679588649 So just to clear up any confusion - the provider hooks exist in a different path in v1-10-stable, so even if I make a change to a provider hook in master, a committer will port

[airflow] branch constraints-master updated: Updating constraints. Build id:222896670

2020-08-24 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-master by this push: new 119281a Updating

[airflow] branch constraints-master updated: Updating constraints. Build id:222896670

2020-08-24 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch constraints-master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/constraints-master by this push: new 119281a Updating

[GitHub] [airflow] mik-laj commented on a change in pull request #9597: [WIP] Add read-only endpoints for task instances

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #9597: URL: https://github.com/apache/airflow/pull/9597#discussion_r476043263 ## File path: airflow/api_connexion/schemas/task_instance_schema.py ## @@ -0,0 +1,147 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[airflow] branch master updated: Improve descriptions in OpenAPI Spec file (#10539)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 49cce8b Improve descriptions in OpenAPI

[GitHub] [airflow] kaxil merged pull request #10539: Improve descriptions in OpenAPI Spec file

2020-08-24 Thread GitBox
kaxil merged pull request #10539: URL: https://github.com/apache/airflow/pull/10539 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[airflow] branch master updated (bfefcce -> c60e476)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from bfefcce Updated REST API call so GET requests pass payload in query string instead of request body (#10462)

[GitHub] [airflow] kaxil merged pull request #10538: Remove mlsd function from hooks/ftp.py

2020-08-24 Thread GitBox
kaxil merged pull request #10538: URL: https://github.com/apache/airflow/pull/10538 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] mik-laj commented on a change in pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10531: URL: https://github.com/apache/airflow/pull/10531#discussion_r475983480 ## File path: docs/scheduler.rst ## @@ -18,11 +18,41 @@ Scheduler == -The Airflow scheduler monitors all tasks and DAGs, then triggers the

[GitHub] [airflow] mik-laj commented on a change in pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10531: URL: https://github.com/apache/airflow/pull/10531#discussion_r475981438 ## File path: docs/scheduler.rst ## @@ -18,11 +18,41 @@ Scheduler == -The Airflow scheduler monitors all tasks and DAGs, then triggers the

[airflow] branch master updated (d1bce91 -> bfefcce)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from d1bce91 PyDocStyle: Enable D403: Capitalized first word of docstring (#10530) add bfefcce Updated REST API

[GitHub] [airflow] potiuk commented on pull request #10462: Updated REST API call so GET requests pass payload in query string instead of request body

2020-08-24 Thread GitBox
potiuk commented on pull request #10462: URL: https://github.com/apache/airflow/pull/10462#issuecomment-679425273 Cool @syudb . What's next ? This is an automated message from the Apache Git Service. To respond to the

[airflow] branch master updated (d1bce91 -> bfefcce)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from d1bce91 PyDocStyle: Enable D403: Capitalized first word of docstring (#10530) add bfefcce Updated REST API

[GitHub] [airflow] mik-laj commented on a change in pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10531: URL: https://github.com/apache/airflow/pull/10531#discussion_r475980600 ## File path: docs/scheduler.rst ## @@ -18,11 +18,41 @@ Scheduler == -The Airflow scheduler monitors all tasks and DAGs, then triggers the

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #10462: Updated REST API call so GET requests pass payload in query string instead of request body

2020-08-24 Thread GitBox
boring-cyborg[bot] commented on pull request #10462: URL: https://github.com/apache/airflow/pull/10462#issuecomment-679424986 Awesome work, congrats on your first merged pull request! This is an automated message from

[GitHub] [airflow] potiuk merged pull request #10462: Updated REST API call so GET requests pass payload in query string instead of request body

2020-08-24 Thread GitBox
potiuk merged pull request #10462: URL: https://github.com/apache/airflow/pull/10462 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] mik-laj commented on a change in pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10531: URL: https://github.com/apache/airflow/pull/10531#discussion_r475977864 ## File path: docs/scheduler.rst ## @@ -18,11 +18,41 @@ Scheduler == -The Airflow scheduler monitors all tasks and DAGs, then triggers the

[GitHub] [airflow] mik-laj commented on pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
mik-laj commented on pull request #10531: URL: https://github.com/apache/airflow/pull/10531#issuecomment-679423694 @yesemsanthoshkumar I would be happy to share information about the scheduler and even provide some internal documents about it so that we have detailed documentation about

[GitHub] [airflow] potiuk commented on pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
potiuk commented on pull request #10531: URL: https://github.com/apache/airflow/pull/10531#issuecomment-679421818 1. Absolutely. 2. Flowchart is great. Ideally, we would just need some sources for that -> for example in Draw.io export format or maybe you would like to recreate the

[airflow] branch master updated (0e0aefb -> d1bce91)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 0e0aefb Fix TestAWSDataSyncOperatorUpdate.__init__ method (#10536) add d1bce91 PyDocStyle: Enable D403:

[GitHub] [airflow] kaxil merged pull request #10530: PyDocStyle: Enable D403: Capitalize first word of docstring

2020-08-24 Thread GitBox
kaxil merged pull request #10530: URL: https://github.com/apache/airflow/pull/10530 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[airflow] branch master updated (be77f8e -> 0e0aefb)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from be77f8e Add a possibility to switch back to building images by secret (#10509) add 0e0aefb Fix

[GitHub] [airflow] kaxil merged pull request #10536: Fix TestAWSDataSyncOperatorUpdate.__init__ method

2020-08-24 Thread GitBox
kaxil merged pull request #10536: URL: https://github.com/apache/airflow/pull/10536 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] kaxil commented on a change in pull request #10539: Improve descriptions in OpenAPI Spec file

2020-08-24 Thread GitBox
kaxil commented on a change in pull request #10539: URL: https://github.com/apache/airflow/pull/10539#discussion_r475965542 ## File path: airflow/api_connexion/openapi/v1.yaml ## @@ -1112,7 +1112,7 @@ paths: /version: get: - summary: Get a version information +

[airflow] branch master updated (6bed074 -> be77f8e)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 6bed074 Remove unreachable code in test_user_command.py (#10526) add be77f8e Add a possibility to switch

[GitHub] [airflow] potiuk merged pull request #10509: Add a possibility to switch back to building images by secret

2020-08-24 Thread GitBox
potiuk merged pull request #10509: URL: https://github.com/apache/airflow/pull/10509 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] mik-laj commented on a change in pull request #10539: Improve descriptions in OpenAPI Spec file

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10539: URL: https://github.com/apache/airflow/pull/10539#discussion_r475963657 ## File path: airflow/api_connexion/openapi/v1.yaml ## @@ -1112,7 +1112,7 @@ paths: /version: get: - summary: Get a version information

[GitHub] [airflow] potiuk commented on pull request #10509: Add a possibility to switch back to building images by secret

2020-08-24 Thread GitBox
potiuk commented on pull request #10509: URL: https://github.com/apache/airflow/pull/10509#issuecomment-679418168 Seems to work @mik-laj ! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] kaxil opened a new pull request #10539: Improve descriptions in OpenAPI Spec file (#10417)

2020-08-24 Thread GitBox
kaxil opened a new pull request #10539: URL: https://github.com/apache/airflow/pull/10539 - Fix various typo for "collection" - Some grammar --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] dimberman commented on a change in pull request #10393: Simplify the K8sExecutor and K8sPodOperator

2020-08-24 Thread GitBox
dimberman commented on a change in pull request #10393: URL: https://github.com/apache/airflow/pull/10393#discussion_r475960847 ## File path: tests/kubernetes/test_pod_generator.py ## @@ -226,238 +186,169 @@ def test_gen_pod_extract_xcom(self, mock_uuid): ],

[airflow] branch master updated (866701c -> 6bed074)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 866701c Fix typo in "Cloud" (#10534) add 6bed074 Remove unreachable code in test_user_command.py (#10526)

[GitHub] [airflow] kaxil merged pull request #10526: Remove unreachable code in test_user_command.py

2020-08-24 Thread GitBox
kaxil merged pull request #10526: URL: https://github.com/apache/airflow/pull/10526 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] dimberman commented on a change in pull request #10393: Simplify the K8sExecutor and K8sPodOperator

2020-08-24 Thread GitBox
dimberman commented on a change in pull request #10393: URL: https://github.com/apache/airflow/pull/10393#discussion_r475959589 ## File path: airflow/providers/cncf/kubernetes/operators/kubernetes_pod.py ## @@ -365,44 +365,57 @@ def create_new_pod_for_operator(self, labels,

[GitHub] [airflow] kaxil opened a new pull request #10538: Remove mlsd function from hooks/ftp.py

2020-08-24 Thread GitBox
kaxil opened a new pull request #10538: URL: https://github.com/apache/airflow/pull/10538 This function was backported for compatibility with Py 2.7. FTP.mlsd was added in Python 3.3 https://docs.python.org/3.6/library/ftplib.html#ftplib.FTP.mlsd --- **^ Add

[airflow] branch master updated (74729d6 -> 866701c)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 74729d6 Fix typo in "Success" (#10537) add 866701c Fix typo in "Cloud" (#10534) No new revisions were

[airflow] branch master updated (bdec5fa -> 74729d6)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from bdec5fa Improving descriptions in OpenAPI (#10417) add 74729d6 Fix typo in "Success" (#10537) No new

[GitHub] [airflow] kaxil merged pull request #10534: Fix typo in "Cloud"

2020-08-24 Thread GitBox
kaxil merged pull request #10534: URL: https://github.com/apache/airflow/pull/10534 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] kaxil merged pull request #10537: Fix typo in "Success"

2020-08-24 Thread GitBox
kaxil merged pull request #10537: URL: https://github.com/apache/airflow/pull/10537 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] potiuk commented on pull request #9650: Helm Docker image sources are now included in the Airlfow codebase

2020-08-24 Thread GitBox
potiuk commented on pull request #9650: URL: https://github.com/apache/airflow/pull/9650#issuecomment-679412668 @kaxil @dimberman @schnie -> I rebased the change where I made all the images used by Helm part of our sources. They are green - and all the sources are easy to rebuild.

[GitHub] [airflow] mik-laj commented on pull request #10537: Fix typo in "Success"

2020-08-24 Thread GitBox
mik-laj commented on pull request #10537: URL: https://github.com/apache/airflow/pull/10537#issuecomment-679411460 Thanks. I didn't like the word either, but when I pasted it in Google, I didn't get a suggestion for the correct spelling, so I found both forms to be correct.

[GitHub] [airflow] kaxil opened a new pull request #10537: Fix typo in "Success"

2020-08-24 Thread GitBox
kaxil opened a new pull request #10537: URL: https://github.com/apache/airflow/pull/10537 "Sucess" -> "Success" --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #10536: Fix TestAWSDataSyncOperatorUpdate.__init__ method

2020-08-24 Thread GitBox
kaxil opened a new pull request #10536: URL: https://github.com/apache/airflow/pull/10536 `__init` -> `__init__` --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] ephraimbuddy opened a new pull request #10535: Add update mask to stable REST API patch dag endpoint

2020-08-24 Thread GitBox
ephraimbuddy opened a new pull request #10535: URL: https://github.com/apache/airflow/pull/10535 This PR adds an update mask to patch dag endpoint. --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #10534: Fix typo in "Cloud"

2020-08-24 Thread GitBox
kaxil opened a new pull request #10534: URL: https://github.com/apache/airflow/pull/10534 `CLoud` -> `Cloud` --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] potiuk commented on pull request #10501: [WIP] Make DagRun and Dag pylint-compatible

2020-08-24 Thread GitBox
potiuk commented on pull request #10501: URL: https://github.com/apache/airflow/pull/10501#issuecomment-679406702 Tomorrow :) This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] potiuk commented on pull request #10501: [WIP] Make DagRun and Dag pylint-compatible

2020-08-24 Thread GitBox
potiuk commented on pull request #10501: URL: https://github.com/apache/airflow/pull/10501#issuecomment-679405606 airflow/providers/amazon/aws/example_dags/example_emr_job_flow_automatic_steps.py:1:0: R0401: Cyclic import (airflow.models.dag -> airflow.operators.subdag_operator)

[GitHub] [airflow] potiuk commented on pull request #10532: Remove pylint filtering

2020-08-24 Thread GitBox
potiuk commented on pull request #10532: URL: https://github.com/apache/airflow/pull/10532#issuecomment-679405333 Next: Black + Type annotations for the core :) This is an automated message from the Apache Git Service. To

[GitHub] [airflow] potiuk commented on pull request #10532: Remove pylint filtering

2020-08-24 Thread GitBox
potiuk commented on pull request #10532: URL: https://github.com/apache/airflow/pull/10532#issuecomment-679403893 Not yet there - we still need to get the DagRun + Dag, but it is awfully close. This is an automated message

[GitHub] [airflow] kaxil opened a new pull request #10533: PyDocStyle: No whitespaces allowed surrounding docstring text

2020-08-24 Thread GitBox
kaxil opened a new pull request #10533: URL: https://github.com/apache/airflow/pull/10533 D210: No whitespaces allowed surrounding docstring text --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil edited a comment on pull request #10532: Remove pylint filtering

2020-08-24 Thread GitBox
kaxil edited a comment on pull request #10532: URL: https://github.com/apache/airflow/pull/10532#issuecomment-679403903 ❤️ --- finally This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] kaxil commented on pull request #10532: Remove pylint filtering

2020-08-24 Thread GitBox
kaxil commented on pull request #10532: URL: https://github.com/apache/airflow/pull/10532#issuecomment-679403903 <3 --- finally This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk opened a new pull request #10532: Remove pylint filtering

2020-08-24 Thread GitBox
potiuk opened a new pull request #10532: URL: https://github.com/apache/airflow/pull/10532 This is a final step of implementing Pylint compatibility, following AIP-6 vote from 11th of April 2019. It took 1.5 yead and 172 commits.

[GitHub] [airflow] potiuk commented on pull request #10501: [WIP] Make DagRun and Dag pylint-compatible

2020-08-24 Thread GitBox
potiuk commented on pull request #10501: URL: https://github.com/apache/airflow/pull/10501#issuecomment-679403362 It will likely fail with cyclic impots we will have to resolve though :( This is an automated message from the

[GitHub] [airflow] yesemsanthoshkumar commented on pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
yesemsanthoshkumar commented on pull request #10531: URL: https://github.com/apache/airflow/pull/10531#issuecomment-679402694 @potiuk Currently a WIP, I'd like you to answer the following questions. 1. IMO, The [old

[GitHub] [airflow] potiuk commented on pull request #10501: [WIP] Make DagRun and Dag pylint-compatible

2020-08-24 Thread GitBox
potiuk commented on pull request #10501: URL: https://github.com/apache/airflow/pull/10501#issuecomment-679402567 This is the last one @BasPH .. https://lists.apache.org/thread.html/f4940d36e98ded96a2473bb2ccdfa4cc648faa2c1334b2aa901c0bba%40%3Cdev.airflow.apache.org%3E

[GitHub] [airflow] yesemsanthoshkumar opened a new pull request #10531: Move scheduler basics documentation to Airflow docs

2020-08-24 Thread GitBox
yesemsanthoshkumar opened a new pull request #10531: URL: https://github.com/apache/airflow/pull/10531 Resolves #10185 --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch master updated: Improving descriptions in OpenAPI (#10417)

2020-08-24 Thread kamilbregula
This is an automated email from the ASF dual-hosted git repository. kamilbregula pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new bdec5fa Improving descriptions in

[GitHub] [airflow] mik-laj merged pull request #10417: Improving descriptions in OpenAPI

2020-08-24 Thread GitBox
mik-laj merged pull request #10417: URL: https://github.com/apache/airflow/pull/10417 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] mik-laj commented on pull request #10509: Add a possibility to switch back to building images by secret

2020-08-24 Thread GitBox
mik-laj commented on pull request #10509: URL: https://github.com/apache/airflow/pull/10509#issuecomment-679397753 I have PR o my fork: https://github.com/mik-laj/airflow/pull/32 This is an automated message from the Apache

[airflow] branch master updated: Fix typo in PostgresHook (#10529)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git The following commit(s) were added to refs/heads/master by this push: new 47265e7 Fix typo in PostgresHook (#10529)

[GitHub] [airflow] kaxil merged pull request #10529: Fix typo in PostgresHook

2020-08-24 Thread GitBox
kaxil merged pull request #10529: URL: https://github.com/apache/airflow/pull/10529 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] kaxil opened a new pull request #10530: PyDocStyle: Enable D403: Capitalized first word of docstring

2020-08-24 Thread GitBox
kaxil opened a new pull request #10530: URL: https://github.com/apache/airflow/pull/10530 First word of the first line should be properly capitalized --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch master updated (1775474 -> 2dab0e2)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 1775474 Make configuration.py Pylint compatible (#10494) add 2dab0e2 Make www/utils.py pylint-compatible

[GitHub] [airflow] potiuk merged pull request #10497: Make www/utils.py pylint-compatible

2020-08-24 Thread GitBox
potiuk merged pull request #10497: URL: https://github.com/apache/airflow/pull/10497 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] mik-laj commented on a change in pull request #10304: WIP Added operator, hook, docs and tests for Dataprep

2020-08-24 Thread GitBox
mik-laj commented on a change in pull request #10304: URL: https://github.com/apache/airflow/pull/10304#discussion_r475926656 ## File path: tests/providers/google/cloud/operators/test_dataprep_system.py ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation (ASF)

[airflow] branch master updated: Make configuration.py Pylint compatible (#10494)

2020-08-24 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.git The following commit(s) were added to refs/heads/master by this push: new 1775474 Make configuration.py Pylint

[GitHub] [airflow] potiuk merged pull request #10494: Make configuration.py Pylint compatible

2020-08-24 Thread GitBox
potiuk merged pull request #10494: URL: https://github.com/apache/airflow/pull/10494 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[airflow] branch master updated (3696c34 -> 4f6d53e)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 3696c34 Fix typo in the word "release" (#10528) add 4f6d53e Make models/taskinstance.py pylint compatible

[GitHub] [airflow] potiuk merged pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
potiuk merged pull request #10499: URL: https://github.com/apache/airflow/pull/10499 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] potiuk commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475924558 ## File path: airflow/sensors/base_sensor_operator.py ## @@ -159,6 +160,16 @@ def _get_next_poke_interval(self, started_at, try_number): else:

[GitHub] [airflow] potiuk commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475924768 ## File path: airflow/sensors/base_sensor_operator.py ## @@ -159,6 +160,16 @@ def _get_next_poke_interval(self, started_at, try_number): else:

[GitHub] [airflow] mik-laj commented on issue #10505: PostgressHook connection - pass additional extra paramters

2020-08-24 Thread GitBox
mik-laj commented on issue #10505: URL: https://github.com/apache/airflow/issues/10505#issuecomment-679392943 @daemon-demon I assigned you to this ticket. This is an automated message from the Apache Git Service. To respond

[GitHub] [airflow] turbaszek commented on a change in pull request #10304: WIP Added operator, hook, docs and tests for Dataprep

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10304: URL: https://github.com/apache/airflow/pull/10304#discussion_r475922760 ## File path: tests/providers/google/cloud/operators/test_dataprep_system.py ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation

[GitHub] [airflow] turbaszek commented on a change in pull request #10304: WIP Added operator, hook, docs and tests for Dataprep

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10304: URL: https://github.com/apache/airflow/pull/10304#discussion_r475922760 ## File path: tests/providers/google/cloud/operators/test_dataprep_system.py ## @@ -0,0 +1,45 @@ +# +# Licensed to the Apache Software Foundation

[airflow] branch master updated (2f2d8db -> 3696c34)

2020-08-24 Thread kaxilnaik
This is an automated email from the ASF dual-hosted git repository. kaxilnaik pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from 2f2d8db Remove all "noinspection" comments native to IntelliJ (#10525) add 3696c34 Fix typo in the word

[GitHub] [airflow] kaxil merged pull request #10528: Fix typo in the word "release"

2020-08-24 Thread GitBox
kaxil merged pull request #10528: URL: https://github.com/apache/airflow/pull/10528 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] daemon-demon commented on issue #10505: PostgressHook connection - pass additional extra paramters

2020-08-24 Thread GitBox
daemon-demon commented on issue #10505: URL: https://github.com/apache/airflow/issues/10505#issuecomment-679391718 Hi, I would like to work on this as my first contribution. This is an automated message from the Apache Git

[GitHub] [airflow] kaxil opened a new pull request #10529: Fix typo in PostgresHook

2020-08-24 Thread GitBox
kaxil opened a new pull request #10529: URL: https://github.com/apache/airflow/pull/10529 `PostrgresHook` -> `PostgresHook` --- **^ Add meaningful description above** Read the **[Pull Request

[airflow] branch master updated (f2da6b4 -> 2f2d8db)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from f2da6b4 Updated documentation for the CI with mermaid sequence diagrams (#10380) add 2f2d8db Remove all

[airflow] branch master updated (f2da6b4 -> 2f2d8db)

2020-08-24 Thread potiuk
This is an automated email from the ASF dual-hosted git repository. potiuk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/airflow.git. from f2da6b4 Updated documentation for the CI with mermaid sequence diagrams (#10380) add 2f2d8db Remove all

[GitHub] [airflow] turbaszek commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475919392 ## File path: airflow/sensors/base_sensor_operator.py ## @@ -159,6 +160,16 @@ def _get_next_poke_interval(self, started_at, try_number):

[GitHub] [airflow] turbaszek commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475919392 ## File path: airflow/sensors/base_sensor_operator.py ## @@ -159,6 +160,16 @@ def _get_next_poke_interval(self, started_at, try_number):

[GitHub] [airflow] potiuk merged pull request #10525: Remove all "noinspection" comments native to IntelliJ

2020-08-24 Thread GitBox
potiuk merged pull request #10525: URL: https://github.com/apache/airflow/pull/10525 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [airflow] turbaszek commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475919005 ## File path: airflow/sensors/base_sensor_operator.py ## @@ -159,6 +160,16 @@ def _get_next_poke_interval(self, started_at, try_number):

[GitHub] [airflow] turbaszek commented on a change in pull request #10494: Make configuration.py Pylint compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10494: URL: https://github.com/apache/airflow/pull/10494#discussion_r475917409 ## File path: airflow/configuration.py ## @@ -540,73 +573,94 @@ def as_dict( :return: Dictionary, where the key is the name of the section

[GitHub] [airflow] kaxil commented on pull request #10527: Covert single quoted docstring to triple

2020-08-24 Thread GitBox
kaxil commented on pull request #10527: URL: https://github.com/apache/airflow/pull/10527#issuecomment-679387258 aah cool, thnx This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] kaxil closed pull request #10527: Covert single quoted docstring to triple

2020-08-24 Thread GitBox
kaxil closed pull request #10527: URL: https://github.com/apache/airflow/pull/10527 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [airflow] potiuk commented on a change in pull request #10499: Make models/taskinstance.py pylint compatible

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10499: URL: https://github.com/apache/airflow/pull/10499#discussion_r475916776 ## File path: airflow/models/taskinstance.py ## @@ -1135,23 +1134,16 @@ def _run_raw_task( session.merge(self) session.commit()

[GitHub] [airflow] potiuk commented on pull request #10527: Covert single quoted docstring to triple

2020-08-24 Thread GitBox
potiuk commented on pull request #10527: URL: https://github.com/apache/airflow/pull/10527#issuecomment-679385832 Just fixed it when removed all noinspections :) This is an automated message from the Apache Git Service. To

[GitHub] [airflow] potiuk commented on a change in pull request #10497: Make www/utils.py pylint-compatible

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10497: URL: https://github.com/apache/airflow/pull/10497#discussion_r475914539 ## File path: airflow/www/utils.py ## @@ -181,7 +181,7 @@ def is_current(current, page): def epoch(dttm): """Returns an epoch-type date""" -

[GitHub] [airflow] mik-laj commented on issue #9080: Build and publish API clients automatically

2020-08-24 Thread GitBox
mik-laj commented on issue #9080: URL: https://github.com/apache/airflow/issues/9080#issuecomment-679383866 We should add info about client library to docs: https://airflow.readthedocs.io/en/latest/stable-rest-api-ref.html

[GitHub] [airflow] kaxil opened a new pull request #10528: Fix typo in the word "release"

2020-08-24 Thread GitBox
kaxil opened a new pull request #10528: URL: https://github.com/apache/airflow/pull/10528 `relase` -> `release` --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] potiuk commented on pull request #10494: Make configuration.py Pylint compatible

2020-08-24 Thread GitBox
potiuk commented on pull request #10494: URL: https://github.com/apache/airflow/pull/10494#issuecomment-679383209 Updated. - should be green now. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] potiuk commented on a change in pull request #10526: Remove unreachable code in test_user_command.py

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10526: URL: https://github.com/apache/airflow/pull/10526#discussion_r475912159 ## File path: tests/cli/commands/test_user_command.py ## @@ -177,7 +177,6 @@ def find_by_username(username): matches = [u for u in

[GitHub] [airflow] potiuk commented on a change in pull request #10526: Remove unreachable code in test_user_command.py

2020-08-24 Thread GitBox
potiuk commented on a change in pull request #10526: URL: https://github.com/apache/airflow/pull/10526#discussion_r475911788 ## File path: tests/cli/commands/test_user_command.py ## @@ -177,7 +177,6 @@ def find_by_username(username): matches = [u for u in

[GitHub] [airflow] kaxil opened a new pull request #10527: Covert single quoted docstring to triple

2020-08-24 Thread GitBox
kaxil opened a new pull request #10527: URL: https://github.com/apache/airflow/pull/10527 As a convention we use Triple quoted docstring --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] kaxil opened a new pull request #10526: Remove unreachable code in test_user_command.py

2020-08-24 Thread GitBox
kaxil opened a new pull request #10526: URL: https://github.com/apache/airflow/pull/10526 `self.fail(` raises exception so it would never run `return None` command --- **^ Add meaningful description above** Read the **[Pull Request

[GitHub] [airflow] turbaszek commented on a change in pull request #10497: Make www/utils.py pylint-compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10497: URL: https://github.com/apache/airflow/pull/10497#discussion_r475909479 ## File path: airflow/www/utils.py ## @@ -181,7 +181,7 @@ def is_current(current, page): def epoch(dttm): """Returns an epoch-type date""" -

[GitHub] [airflow] turbaszek commented on a change in pull request #10497: Make www/utils.py pylint-compatible

2020-08-24 Thread GitBox
turbaszek commented on a change in pull request #10497: URL: https://github.com/apache/airflow/pull/10497#discussion_r475909479 ## File path: airflow/www/utils.py ## @@ -181,7 +181,7 @@ def is_current(current, page): def epoch(dttm): """Returns an epoch-type date""" -

  1   2   3   4   >