[GitHub] XD-DENG commented on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test

2018-10-13 Thread GitBox
XD-DENG commented on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test URL: https://github.com/apache/incubator-airflow/pull/4049#issuecomment-429591267 Hi @Fokko, As we discussed in the email list yesterday, some scripts in `/test` should be renamed to ensure they can be

[GitHub] codecov-io edited a comment on issue #3683: [AIRFLOW-2770] kubernetes: add support for dag folder in the docker i…

2018-10-13 Thread GitBox
codecov-io edited a comment on issue #3683: [AIRFLOW-2770] kubernetes: add support for dag folder in the docker i… URL: https://github.com/apache/incubator-airflow/pull/3683#issuecomment-409915837 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3683?src=pr=h1) Report

[GitHub] bkvarda commented on issue #4008: [AIRFLOW-3155] Add ability to filter by a last modified time in GoogleCloudStorageToGoogleCloudStorageOperator

2018-10-13 Thread GitBox
bkvarda commented on issue #4008: [AIRFLOW-3155] Add ability to filter by a last modified time in GoogleCloudStorageToGoogleCloudStorageOperator URL: https://github.com/apache/incubator-airflow/pull/4008#issuecomment-429572002 @Fokko I agree - I’ll make that change

[GitHub] sbilinski commented on issue #3914: [AIRFLOW-3069] Log all output of the S3 file transform script

2018-10-13 Thread GitBox
sbilinski commented on issue #3914: [AIRFLOW-3069] Log all output of the S3 file transform script URL: https://github.com/apache/incubator-airflow/pull/3914#issuecomment-429556983 @Fokko I've applied some changes and rebased against `master` - please take another look.

[GitHub] codecov-io edited a comment on issue #3914: [AIRFLOW-3069] Log all output of the S3 file transform script

2018-10-13 Thread GitBox
codecov-io edited a comment on issue #3914: [AIRFLOW-3069] Log all output of the S3 file transform script URL: https://github.com/apache/incubator-airflow/pull/3914#issuecomment-422548293 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/3914?src=pr=h1) Report > Merging

[jira] [Updated] (AIRFLOW-3203) Bugs in DockerOperator & its test

2018-10-13 Thread Xiaodong DENG (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3203?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Xiaodong DENG updated AIRFLOW-3203: --- Affects Version/s: 1.10.0 > Bugs in DockerOperator & its test >

[GitHub] codecov-io edited a comment on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test

2018-10-13 Thread GitBox
codecov-io edited a comment on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test URL: https://github.com/apache/incubator-airflow/pull/4049#issuecomment-429554020 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4049?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test

2018-10-13 Thread GitBox
codecov-io commented on issue #4049: [AIRFLOW-3203] Fix DockerOperator & its test URL: https://github.com/apache/incubator-airflow/pull/4049#issuecomment-429554020 # [Codecov](https://codecov.io/gh/apache/incubator-airflow/pull/4049?src=pr=h1) Report > Merging

[GitHub] chandulal commented on issue #4046: Revert: Need some approval to add name in companies list.

2018-10-13 Thread GitBox
chandulal commented on issue #4046: Revert: Need some approval to add name in companies list. URL: https://github.com/apache/incubator-airflow/pull/4046#issuecomment-429552932 @Fokko There is only one change in README.md. Just removed "Grab" from company list. Because we need to take

[jira] [Commented] (AIRFLOW-3203) Bugs in DockerOperator & its test

2018-10-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648991#comment-16648991 ] ASF GitHub Bot commented on AIRFLOW-3203: - XD-DENG opened a new pull request #4049:

[GitHub] XD-DENG opened a new pull request #4049: [AIRFLOW-3203] Fix DockerOperator & its test

2018-10-13 Thread GitBox
XD-DENG opened a new pull request #4049: [AIRFLOW-3203] Fix DockerOperator & its test URL: https://github.com/apache/incubator-airflow/pull/4049 ### Jira - https://issues.apache.org/jira/browse/AIRFLOW-3203 ### Description - For argument `image`, no need to

[jira] [Created] (AIRFLOW-3203) Bugs in DockerOperator & its test

2018-10-13 Thread Xiaodong DENG (JIRA)
Xiaodong DENG created AIRFLOW-3203: -- Summary: Bugs in DockerOperator & its test Key: AIRFLOW-3203 URL: https://issues.apache.org/jira/browse/AIRFLOW-3203 Project: Apache Airflow Issue Type:

[GitHub] sid88in commented on issue #4048: [AIRFLOW-3202] add missing documentation for AWS hooks/operator

2018-10-13 Thread GitBox
sid88in commented on issue #4048: [AIRFLOW-3202] add missing documentation for AWS hooks/operator URL: https://github.com/apache/incubator-airflow/pull/4048#issuecomment-429518493 thanks @Fokko !! This is an automated

[jira] [Resolved] (AIRFLOW-3202) Update Airflow Docs with missing AWS hooks/operators

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong resolved AIRFLOW-3202. --- Resolution: Fixed > Update Airflow Docs with missing AWS hooks/operators >

[GitHub] Fokko closed pull request #4048: [AIRFLOW-3202] add missing documentation for AWS hooks/operator

2018-10-13 Thread GitBox
Fokko closed pull request #4048: [AIRFLOW-3202] add missing documentation for AWS hooks/operator URL: https://github.com/apache/incubator-airflow/pull/4048 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[jira] [Commented] (AIRFLOW-3202) Update Airflow Docs with missing AWS hooks/operators

2018-10-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648790#comment-16648790 ] ASF GitHub Bot commented on AIRFLOW-3202: - Fokko closed pull request #4048: [AIRFLOW-3202] add

[jira] [Updated] (AIRFLOW-3202) Update Airflow Docs with missing AWS hooks/operators

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong updated AIRFLOW-3202: -- Fix Version/s: 2.0.0 > Update Airflow Docs with missing AWS hooks/operators >

[jira] [Resolved] (AIRFLOW-3174) Refine Docstring for SQL Operators

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3174?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong resolved AIRFLOW-3174. --- Resolution: Fixed Fix Version/s: 2.0.0 > Refine Docstring for SQL Operators

[GitHub] Fokko commented on a change in pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API

2018-10-13 Thread GitBox
Fokko commented on a change in pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API URL: https://github.com/apache/incubator-airflow/pull/4044#discussion_r224951208 ## File path: UPDATING.md ## @@ -72,6 +72,16 @@ then you need to change it

[jira] [Commented] (AIRFLOW-3197) EMR Hook is missing some parameters to valid on the AWS API

2018-10-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648772#comment-16648772 ] ASF GitHub Bot commented on AIRFLOW-3197: - Fokko closed pull request #4044: [AIRFLOW-3197]

[GitHub] Fokko closed pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API

2018-10-13 Thread GitBox
Fokko closed pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API URL: https://github.com/apache/incubator-airflow/pull/4044 This is an automated message from the Apache Git Service. To

[GitHub] Fokko commented on a change in pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API

2018-10-13 Thread GitBox
Fokko commented on a change in pull request #4044: [AIRFLOW-3197] EMRHook is missing new parameters of the AWS API URL: https://github.com/apache/incubator-airflow/pull/4044#discussion_r224951208 ## File path: UPDATING.md ## @@ -72,6 +72,16 @@ then you need to change it

[jira] [Resolved] (AIRFLOW-3197) EMR Hook is missing some parameters to valid on the AWS API

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong resolved AIRFLOW-3197. --- Resolution: Fixed > EMR Hook is missing some parameters to valid on the AWS API >

[jira] [Assigned] (AIRFLOW-3197) EMR Hook is missing some parameters to valid on the AWS API

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong reassigned AIRFLOW-3197: - Assignee: Fokko Driesprong > EMR Hook is missing some parameters to valid on

[jira] [Assigned] (AIRFLOW-3197) EMR Hook is missing some parameters to valid on the AWS API

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3197?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong reassigned AIRFLOW-3197: - Assignee: Ash Berlin-Taylor (was: Fokko Driesprong) > EMR Hook is missing

[GitHub] Fokko edited a comment on issue #4003: [AIRFLOW-3163] add operator to enable setting table description in BigQuery table

2018-10-13 Thread GitBox
Fokko edited a comment on issue #4003: [AIRFLOW-3163] add operator to enable setting table description in BigQuery table URL: https://github.com/apache/incubator-airflow/pull/4003#issuecomment-429515036 Just thinking along here. We could extend the create table definition and add the

[GitHub] Fokko commented on issue #4003: [AIRFLOW-3163] add operator to enable setting table description in BigQuery table

2018-10-13 Thread GitBox
Fokko commented on issue #4003: [AIRFLOW-3163] add operator to enable setting table description in BigQuery table URL: https://github.com/apache/incubator-airflow/pull/4003#issuecomment-429515036 Just thinking along here. We could extend the create table definition and add the

[GitHub] XD-DENG commented on issue #4043: [AIRFLOW-3174] Refine Docstring for SQL Operators & Hooks

2018-10-13 Thread GitBox
XD-DENG commented on issue #4043: [AIRFLOW-3174] Refine Docstring for SQL Operators & Hooks URL: https://github.com/apache/incubator-airflow/pull/4043#issuecomment-429514750 Thanks @Fokko for merging :-) This is an

[GitHub] Fokko closed pull request #4043: [AIRFLOW-3174] Refine Docstring for SQL Operators & Hooks

2018-10-13 Thread GitBox
Fokko closed pull request #4043: [AIRFLOW-3174] Refine Docstring for SQL Operators & Hooks URL: https://github.com/apache/incubator-airflow/pull/4043 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[jira] [Commented] (AIRFLOW-3174) Refine Docstring for SQL Operators

2018-10-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648767#comment-16648767 ] ASF GitHub Bot commented on AIRFLOW-3174: - Fokko closed pull request #4043: [AIRFLOW-3174]

[jira] [Resolved] (AIRFLOW-2932) GoogleCloudStorageHook - allow compression of file

2018-10-13 Thread Fokko Driesprong (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2932?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fokko Driesprong resolved AIRFLOW-2932. --- Resolution: Fixed Fix Version/s: 1.10.1 > GoogleCloudStorageHook - allow

[jira] [Commented] (AIRFLOW-2932) GoogleCloudStorageHook - allow compression of file

2018-10-13 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-2932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648765#comment-16648765 ] ASF GitHub Bot commented on AIRFLOW-2932: - Fokko closed pull request #3893: [AIRFLOW-2932]

[GitHub] Fokko closed pull request #3893: [AIRFLOW-2932] GoogleCloudStorageHook - allow compression of file

2018-10-13 Thread GitBox
Fokko closed pull request #3893: [AIRFLOW-2932] GoogleCloudStorageHook - allow compression of file URL: https://github.com/apache/incubator-airflow/pull/3893 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] Fokko commented on issue #3893: [AIRFLOW-2932] GoogleCloudStorageHook - allow compression of file

2018-10-13 Thread GitBox
Fokko commented on issue #3893: [AIRFLOW-2932] GoogleCloudStorageHook - allow compression of file URL: https://github.com/apache/incubator-airflow/pull/3893#issuecomment-429514276 Thanks @neil90 @kaxil. This is an

[GitHub] Fokko commented on issue #4008: [AIRFLOW-3155] Add ability to filter by a last modified time in GoogleCloudStorageToGoogleCloudStorageOperator

2018-10-13 Thread GitBox
Fokko commented on issue #4008: [AIRFLOW-3155] Add ability to filter by a last modified time in GoogleCloudStorageToGoogleCloudStorageOperator URL: https://github.com/apache/incubator-airflow/pull/4008#issuecomment-429514162 @bkvarda I have one more thought, let me know what you think of