[GitHub] [airflow] dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook

dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook URL: https://github.com/apache/airflow/pull/6104#discussion_r324483395 ## File path: airflow/contrib/hooks/ssh_hook.py ## @@ -160,24 +169,27 @@ def get_conn(self):

[GitHub] [airflow] dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook

dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook URL: https://github.com/apache/airflow/pull/6104#discussion_r324483395 ## File path: airflow/contrib/hooks/ssh_hook.py ## @@ -160,24 +169,27 @@ def get_conn(self):

[jira] [Comment Edited] (AIRFLOW-5447) KubernetesExecutor hangs on task queueing

[ https://issues.apache.org/jira/browse/AIRFLOW-5447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930224#comment-16930224 ] Daniel Imberman edited comment on AIRFLOW-5447 at 9/16/19 4:21 AM: ---

[jira] [Commented] (AIRFLOW-5447) KubernetesExecutor hangs on task queueing

[ https://issues.apache.org/jira/browse/AIRFLOW-5447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930224#comment-16930224 ] Daniel Imberman commented on AIRFLOW-5447: -- [~Yuval.Itzchakov] [~cwegrzyn]Thank you guys for

[GitHub] [airflow] jasonlin2268 opened a new pull request #6116: Merge pull request #1 from apache/master

jasonlin2268 opened a new pull request #6116: Merge pull request #1 from apache/master URL: https://github.com/apache/airflow/pull/6116 fix Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930177#comment-16930177 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930174#comment-16930174 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[GitHub] [airflow] potiuk commented on issue #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes

potiuk commented on issue #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes URL: https://github.com/apache/airflow/pull/5786#issuecomment-531613726 @feluelle -> I think it will be green soon:)

[GitHub] [airflow] potiuk commented on a change in pull request #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes

potiuk commented on a change in pull request #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes URL: https://github.com/apache/airflow/pull/5786#discussion_r324488969 ## File path: airflow/kubernetes/volume.py ## @@ -25,20

[GitHub] [airflow] potiuk commented on a change in pull request #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes

potiuk commented on a change in pull request #5786: [AIRFLOW-5170] [AIRFLOW-5256] Consistent licences for python files and related pylint fixes URL: https://github.com/apache/airflow/pull/5786#discussion_r324488944 ## File path: airflow/kubernetes/volume.py ## @@ -25,20

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930170#comment-16930170 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[GitHub] [airflow] potiuk commented on issue #5976: [AIRFLOW-5369] Add interactivity to pre-commits

potiuk commented on issue #5976: [AIRFLOW-5369] Add interactivity to pre-commits URL: https://github.com/apache/airflow/pull/5976#issuecomment-531610072 Hey @dimberman :)? This is an automated message from the Apache Git

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930168#comment-16930168 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930167#comment-16930167 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[GitHub] [airflow] potiuk commented on issue #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on issue #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#issuecomment-531608665 Hey @gerardo -> I think maybe you can take a look at running the dind image separately. It would be simplification and it will make the

[GitHub] [airflow] potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#discussion_r324486431 ## File path: scripts/ci/local_ci_stop_environment.sh ## @@ -45,7 +45,6 @@ export HOST_GROUP_ID

[GitHub] [airflow] potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#discussion_r324486402 ## File path: scripts/ci/pylint_todo.txt ## @@ -462,8 +462,8 @@ ./tests/lineage/backend/test_atlas.py

[GitHub] [airflow] potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#discussion_r324486346 ## File path: Dockerfile ## @@ -85,6 +92,24 @@ RUN curl -sL https://deb.nodesource.com/setup_10.x | bash

[GitHub] [airflow] potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#discussion_r324486249 ## File path: Dockerfile ## @@ -372,4 +397,6 @@ EXPOSE 8080 ENTRYPOINT ["/usr/local/bin/dumb-init",

[GitHub] [airflow] potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI

potiuk commented on a change in pull request #5837: [AIRFLOW-5223] Use KinD for Kubernetes in CI URL: https://github.com/apache/airflow/pull/5837#discussion_r324486045 ## File path: scripts/ci/kubernetes/docker/build.sh ## @@ -20,29 +20,14 @@ IMAGE=${IMAGE:-airflow}

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930163#comment-16930163 ] ASF GitHub Bot commented on AIRFLOW-5150: - potiuk commented on pull request #6115:

[GitHub] [airflow] potiuk opened a new pull request #6115: [AIRFLOW-5150] Implement POC for GitLab + Kubernetes tests

potiuk opened a new pull request #6115: [AIRFLOW-5150] Implement POC for GitLab + Kubernetes tests URL: https://github.com/apache/airflow/pull/6115 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[jira] [Resolved] (AIRFLOW-5476) Typo in BREEZE.rst

[ https://issues.apache.org/jira/browse/AIRFLOW-5476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lee Dongjin resolved AIRFLOW-5476. -- Resolution: Fixed > Typo in BREEZE.rst > -- > > Key:

[jira] [Commented] (AIRFLOW-5487) airflow/utils/strings.py: Fix unused warning variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930154#comment-16930154 ] ASF subversion and git services commented on AIRFLOW-5487: -- Commit

[jira] [Commented] (AIRFLOW-5488) Remove unused variables from tmp_configuration_copy method

[ https://issues.apache.org/jira/browse/AIRFLOW-5488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930156#comment-16930156 ] ASF subversion and git services commented on AIRFLOW-5488: -- Commit

[jira] [Commented] (AIRFLOW-5489) bash_senor: Remove unneeded assignment of variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930151#comment-16930151 ] ASF subversion and git services commented on AIRFLOW-5489: -- Commit

[jira] [Commented] (AIRFLOW-5496) hive_stats_operator: Unneeded variable assignment

[ https://issues.apache.org/jira/browse/AIRFLOW-5496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930153#comment-16930153 ] ASF subversion and git services commented on AIRFLOW-5496: -- Commit

[jira] [Commented] (AIRFLOW-5492) Missing docstring for hive .py

[ https://issues.apache.org/jira/browse/AIRFLOW-5492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930152#comment-16930152 ] ASF subversion and git services commented on AIRFLOW-5492: -- Commit

[jira] [Commented] (AIRFLOW-5490) security.py: Fix incorrect None comparison

[ https://issues.apache.org/jira/browse/AIRFLOW-5490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930150#comment-16930150 ] ASF subversion and git services commented on AIRFLOW-5490: -- Commit

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930158#comment-16930158 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[jira] [Commented] (AIRFLOW-5484) PigCliHook has incorrect named parameter

[ https://issues.apache.org/jira/browse/AIRFLOW-5484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930155#comment-16930155 ] ASF subversion and git services commented on AIRFLOW-5484: -- Commit

[jira] [Commented] (AIRFLOW-5493) cli.py has unnecessary paren wrapping of scala variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930157#comment-16930157 ] ASF subversion and git services commented on AIRFLOW-5493: -- Commit

[jira] [Commented] (AIRFLOW-5482) Deprecate Schedule Interval on task level

[ https://issues.apache.org/jira/browse/AIRFLOW-5482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930147#comment-16930147 ] ASF subversion and git services commented on AIRFLOW-5482: -- Commit

[jira] [Commented] (AIRFLOW-5465) Fix deprecated imports in examples

[ https://issues.apache.org/jira/browse/AIRFLOW-5465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930141#comment-16930141 ] ASF subversion and git services commented on AIRFLOW-5465: -- Commit

[jira] [Commented] (AIRFLOW-5464) Fix GCP Memorystore example DAG

[ https://issues.apache.org/jira/browse/AIRFLOW-5464?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930142#comment-16930142 ] ASF subversion and git services commented on AIRFLOW-5464: -- Commit

[jira] [Commented] (AIRFLOW-5482) Deprecate Schedule Interval on task level

[ https://issues.apache.org/jira/browse/AIRFLOW-5482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930148#comment-16930148 ] ASF subversion and git services commented on AIRFLOW-5482: -- Commit

[jira] [Commented] (AIRFLOW-5479) Normalize get_conn method in GCP Kubernetes hook

[ https://issues.apache.org/jira/browse/AIRFLOW-5479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930146#comment-16930146 ] ASF subversion and git services commented on AIRFLOW-5479: -- Commit

[jira] [Commented] (AIRFLOW-5476) Typo in BREEZE.rst

[ https://issues.apache.org/jira/browse/AIRFLOW-5476?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930143#comment-16930143 ] ASF subversion and git services commented on AIRFLOW-5476: -- Commit

[jira] [Commented] (AIRFLOW-5471) Fix docstring in GcpTransferServiceOperationsListOperator

[ https://issues.apache.org/jira/browse/AIRFLOW-5471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930145#comment-16930145 ] ASF subversion and git services commented on AIRFLOW-5471: -- Commit

[jira] [Commented] (AIRFLOW-5480) Fix flaky impersonation test

[ https://issues.apache.org/jira/browse/AIRFLOW-5480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930144#comment-16930144 ] ASF subversion and git services commented on AIRFLOW-5480: -- Commit

[jira] [Commented] (AIRFLOW-5150) Implement POC of GitLab CI + GKE integration

[ https://issues.apache.org/jira/browse/AIRFLOW-5150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930149#comment-16930149 ] ASF subversion and git services commented on AIRFLOW-5150: -- Commit

[GitHub] [airflow] mik-laj merged pull request #6087: [AIRFLOW-XXX][depends on #6086] Display AWS integrations in table

mik-laj merged pull request #6087: [AIRFLOW-XXX][depends on #6086] Display AWS integrations in table URL: https://github.com/apache/airflow/pull/6087 This is an automated message from the Apache Git Service. To respond to

[jira] [Commented] (AIRFLOW-5493) cli.py has unnecessary paren wrapping of scala variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930136#comment-16930136 ] ASF GitHub Bot commented on AIRFLOW-5493: - jghoman commented on pull request #6113:

[jira] [Commented] (AIRFLOW-5493) cli.py has unnecessary paren wrapping of scala variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930137#comment-16930137 ] ASF subversion and git services commented on AIRFLOW-5493: -- Commit

[GitHub] [airflow] jghoman merged pull request #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable

jghoman merged pull request #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable URL: https://github.com/apache/airflow/pull/6113 This is an automated message from the Apache Git Service. To

[GitHub] [airflow] jghoman commented on issue #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable

jghoman commented on issue #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable URL: https://github.com/apache/airflow/pull/6113#issuecomment-531603686 +1. Looks good, Manuel! Thanks for the contribution and welcome to Airflow.

[GitHub] [airflow] dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook

dstandish commented on a change in pull request #6104: [AIRFLOW-4574] allow providing private_key in SSHHook URL: https://github.com/apache/airflow/pull/6104#discussion_r324483395 ## File path: airflow/contrib/hooks/ssh_hook.py ## @@ -160,24 +169,27 @@ def get_conn(self):

[jira] [Commented] (AIRFLOW-5488) Remove unused variables from tmp_configuration_copy method

[ https://issues.apache.org/jira/browse/AIRFLOW-5488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930134#comment-16930134 ] ASF GitHub Bot commented on AIRFLOW-5488: - jghoman commented on pull request #6114:

[jira] [Commented] (AIRFLOW-5488) Remove unused variables from tmp_configuration_copy method

[ https://issues.apache.org/jira/browse/AIRFLOW-5488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930135#comment-16930135 ] ASF subversion and git services commented on AIRFLOW-5488: -- Commit

[GitHub] [airflow] jghoman commented on issue #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method

jghoman commented on issue #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method URL: https://github.com/apache/airflow/pull/6114#issuecomment-531603270 +1. Looks great, Fernando! Thanks for the contribution and welcome to Airflow!

[GitHub] [airflow] jghoman merged pull request #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method

jghoman merged pull request #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method URL: https://github.com/apache/airflow/pull/6114 This is an automated message from the Apache Git Service. To

[GitHub] [airflow] FernandoMon commented on issue #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method

FernandoMon commented on issue #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method URL: https://github.com/apache/airflow/pull/6114#issuecomment-531603239 @jghoman This is an automated message

[jira] [Commented] (AIRFLOW-5484) PigCliHook has incorrect named parameter

[ https://issues.apache.org/jira/browse/AIRFLOW-5484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930132#comment-16930132 ] ASF GitHub Bot commented on AIRFLOW-5484: - jghoman commented on pull request #6112:

[jira] [Commented] (AIRFLOW-5484) PigCliHook has incorrect named parameter

[ https://issues.apache.org/jira/browse/AIRFLOW-5484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930133#comment-16930133 ] ASF subversion and git services commented on AIRFLOW-5484: -- Commit

[GitHub] [airflow] jghoman merged pull request #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter

jghoman merged pull request #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter URL: https://github.com/apache/airflow/pull/6112 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] jghoman commented on issue #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter

jghoman commented on issue #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter URL: https://github.com/apache/airflow/pull/6112#issuecomment-531603141 +1. Looks great, Josue! Thanks for the contribution, welcome to Airflow!

[jira] [Commented] (AIRFLOW-5488) Remove unused variables from tmp_configuration_copy method

[ https://issues.apache.org/jira/browse/AIRFLOW-5488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930131#comment-16930131 ] ASF GitHub Bot commented on AIRFLOW-5488: - FernandoMon commented on pull request #6114:

[GitHub] [airflow] FernandoMon opened a new pull request #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method

FernandoMon opened a new pull request #6114: [AIRFLOW-5488]Remove unused variables from tmp_configuration_copy method URL: https://github.com/apache/airflow/pull/6114 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] [airflow] soldraghz commented on issue #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable

soldraghz commented on issue #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable URL: https://github.com/apache/airflow/pull/6113#issuecomment-531602900 @jghoman hey can you review this? thanks

[jira] [Commented] (AIRFLOW-5493) cli.py has unnecessary paren wrapping of scala variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930130#comment-16930130 ] ASF GitHub Bot commented on AIRFLOW-5493: - soldraghz commented on pull request #6113:

[GitHub] [airflow] soldraghz opened a new pull request #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable

soldraghz opened a new pull request #6113: [AIRFLOW-5493]: fix cli.py unnecessary paren wrapping of scala variable URL: https://github.com/apache/airflow/pull/6113 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] [airflow] yorologo commented on issue #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter

yorologo commented on issue #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter URL: https://github.com/apache/airflow/pull/6112#issuecomment-531602674 @jghoman Thanks for this experience This is an automated

[jira] [Commented] (AIRFLOW-5484) PigCliHook has incorrect named parameter

[ https://issues.apache.org/jira/browse/AIRFLOW-5484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930127#comment-16930127 ] ASF GitHub Bot commented on AIRFLOW-5484: - yorologo commented on pull request #6112:

[GitHub] [airflow] yorologo opened a new pull request #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter

yorologo opened a new pull request #6112: AIRFLOW-5484: fix PigCliHook incorrectly named parameter URL: https://github.com/apache/airflow/pull/6112 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5487) airflow/utils/strings.py: Fix unused warning variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930125#comment-16930125 ] ASF subversion and git services commented on AIRFLOW-5487: -- Commit

[GitHub] [airflow] aijamalnk merged pull request #6111: [AIRFLOW-5487]Fix unused warning var

aijamalnk merged pull request #6111: [AIRFLOW-5487]Fix unused warning var URL: https://github.com/apache/airflow/pull/6111 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Commented] (AIRFLOW-5487) airflow/utils/strings.py: Fix unused warning variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930123#comment-16930123 ] ASF GitHub Bot commented on AIRFLOW-5487: - aijamalnk commented on pull request #6111:

[GitHub] [airflow] mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#discussion_r324482337 ## File path: airflow/www/security.py ## @@ -340,8 +340,8 @@ def clean_perms(self): pvms = (

[GitHub] [airflow] jghoman commented on issue #6111: [AIRFLOW-5487]Fix unused warning var

jghoman commented on issue #6111: [AIRFLOW-5487]Fix unused warning var URL: https://github.com/apache/airflow/pull/6111#issuecomment-531602140 +1. Looks great. Thanks, Adan! This is an automated message from the Apache Git

[jira] [Commented] (AIRFLOW-5496) hive_stats_operator: Unneeded variable assignment

[ https://issues.apache.org/jira/browse/AIRFLOW-5496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930122#comment-16930122 ] ASF subversion and git services commented on AIRFLOW-5496: -- Commit

[jira] [Commented] (AIRFLOW-5496) hive_stats_operator: Unneeded variable assignment

[ https://issues.apache.org/jira/browse/AIRFLOW-5496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930121#comment-16930121 ] ASF GitHub Bot commented on AIRFLOW-5496: - jghoman commented on pull request #6110:

[GitHub] [airflow] jghoman merged pull request #6110: [AIRFLOW-5496] Delete unneeded variable assignment

jghoman merged pull request #6110: [AIRFLOW-5496] Delete unneeded variable assignment URL: https://github.com/apache/airflow/pull/6110 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] jghoman commented on issue #6110: [AIRFLOW-5496] Delete unneeded variable assignment

jghoman commented on issue #6110: [AIRFLOW-5496] Delete unneeded variable assignment URL: https://github.com/apache/airflow/pull/6110#issuecomment-531602023 +1. Looks great. Thanks for the contribution, Daniela! Welcome to Airflow.

[jira] [Commented] (AIRFLOW-5487) airflow/utils/strings.py: Fix unused warning variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930119#comment-16930119 ] ASF GitHub Bot commented on AIRFLOW-5487: - adankro commented on pull request #6111:

[GitHub] [airflow] adankro commented on issue #6111: [AIRFLOW-5487]Fix unused warning var

adankro commented on issue #6111: [AIRFLOW-5487]Fix unused warning var URL: https://github.com/apache/airflow/pull/6111#issuecomment-531601991 @jghoman This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] daniela-BV commented on issue #6110: [AIRFLOW-5496] Delete unneeded variable assignment

daniela-BV commented on issue #6110: [AIRFLOW-5496] Delete unneeded variable assignment URL: https://github.com/apache/airflow/pull/6110#issuecomment-531601942 @jghoman Hey, Can you review this? Thank you for your time This

[GitHub] [airflow] adankro opened a new pull request #6111: [AIRFLOW-5487]Fix unused warning var

adankro opened a new pull request #6111: [AIRFLOW-5487]Fix unused warning var URL: https://github.com/apache/airflow/pull/6111 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5496) hive_stats_operator: Unneeded variable assignment

[ https://issues.apache.org/jira/browse/AIRFLOW-5496?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930118#comment-16930118 ] ASF GitHub Bot commented on AIRFLOW-5496: - daniela-BV commented on pull request #6110:

[GitHub] [airflow] daniela-BV opened a new pull request #6110: [AIRFLOW-5496] Delete unneeded variable assignment

daniela-BV opened a new pull request #6110: [AIRFLOW-5496] Delete unneeded variable assignment URL: https://github.com/apache/airflow/pull/6110 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5492) Missing docstring for hive .py

[ https://issues.apache.org/jira/browse/AIRFLOW-5492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930117#comment-16930117 ] ASF subversion and git services commented on AIRFLOW-5492: -- Commit

[jira] [Commented] (AIRFLOW-5492) Missing docstring for hive .py

[ https://issues.apache.org/jira/browse/AIRFLOW-5492?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930116#comment-16930116 ] ASF GitHub Bot commented on AIRFLOW-5492: - jghoman commented on pull request #6107:

[jira] [Commented] (AIRFLOW-5489) bash_senor: Remove unneeded assignment of variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930115#comment-16930115 ] ASF subversion and git services commented on AIRFLOW-5489: -- Commit

[GitHub] [airflow] jghoman merged pull request #6107: [AIRFLOW-5492] added missing docstrings

jghoman merged pull request #6107: [AIRFLOW-5492] added missing docstrings URL: https://github.com/apache/airflow/pull/6107 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] jghoman commented on issue #6107: [AIRFLOW-5492] added missing docstrings

jghoman commented on issue #6107: [AIRFLOW-5492] added missing docstrings URL: https://github.com/apache/airflow/pull/6107#issuecomment-531601765 +1. Thanks for the contribution, Pedro! Welcome to Airflow. This is an

[GitHub] [airflow] aijamalnk merged pull request #6106: [AIRFLOW-5489] Remove unneeded assignment of variable

aijamalnk merged pull request #6106: [AIRFLOW-5489] Remove unneeded assignment of variable URL: https://github.com/apache/airflow/pull/6106 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] aijamalnk commented on issue #6106: [AIRFLOW-5489] Remove unneeded assignment of variable

aijamalnk commented on issue #6106: [AIRFLOW-5489] Remove unneeded assignment of variable URL: https://github.com/apache/airflow/pull/6106#issuecomment-531601692 LGTM:) Thank you for your contribution Gerardo! This is an

[GitHub] [airflow] jghoman commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

jghoman commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#discussion_r324482305 ## File path: airflow/www/security.py ## @@ -340,8 +340,8 @@ def clean_perms(self): pvms = (

[jira] [Commented] (AIRFLOW-5489) bash_senor: Remove unneeded assignment of variable

[ https://issues.apache.org/jira/browse/AIRFLOW-5489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930114#comment-16930114 ] ASF GitHub Bot commented on AIRFLOW-5489: - aijamalnk commented on pull request #6106:

[GitHub] [airflow] mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#discussion_r324482268 ## File path: airflow/www/security.py ## @@ -340,8 +340,8 @@ def clean_perms(self): pvms = (

[GitHub] [airflow] mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#discussion_r324482337 ## File path: airflow/www/security.py ## @@ -340,8 +340,8 @@ def clean_perms(self): pvms = (

[GitHub] [airflow] jghoman commented on issue #6109: [AIRFLOW-5490] Fix incorrect None comparison

jghoman commented on issue #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#issuecomment-531601600 +1. Thanks for the contribution, Alexis! Welcome to Airflow! This is an

[GitHub] [airflow] jghoman merged pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

jghoman merged pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109 This is an automated message from the Apache Git Service. To respond to the message, please log

[jira] [Commented] (AIRFLOW-5490) security.py: Fix incorrect None comparison

[ https://issues.apache.org/jira/browse/AIRFLOW-5490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930113#comment-16930113 ] ASF subversion and git services commented on AIRFLOW-5490: -- Commit

[jira] [Commented] (AIRFLOW-5490) security.py: Fix incorrect None comparison

[ https://issues.apache.org/jira/browse/AIRFLOW-5490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930112#comment-16930112 ] ASF GitHub Bot commented on AIRFLOW-5490: - jghoman commented on pull request #6109:

[GitHub] [airflow] mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

mik-laj commented on a change in pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109#discussion_r324482268 ## File path: airflow/www/security.py ## @@ -340,8 +340,8 @@ def clean_perms(self): pvms = (

[jira] [Commented] (AIRFLOW-5490) security.py: Fix incorrect None comparison

[ https://issues.apache.org/jira/browse/AIRFLOW-5490?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930109#comment-16930109 ] ASF GitHub Bot commented on AIRFLOW-5490: - Alexis-Gu-Ar commented on pull request #6109:

[GitHub] [airflow] Alexis-Gu-Ar opened a new pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison

Alexis-Gu-Ar opened a new pull request #6109: [AIRFLOW-5490] Fix incorrect None comparison URL: https://github.com/apache/airflow/pull/6109 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[jira] [Commented] (AIRFLOW-5491) mark_tasks pydoc is incorrect

[ https://issues.apache.org/jira/browse/AIRFLOW-5491?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16930108#comment-16930108 ] ASF subversion and git services commented on AIRFLOW-5491: -- Commit

[GitHub] [airflow] Mordeck commented on issue #6108: [AIRFLOW-5491] mark_tasks pydoc is incorrect

Mordeck commented on issue #6108: [AIRFLOW-5491] mark_tasks pydoc is incorrect URL: https://github.com/apache/airflow/pull/6108#issuecomment-531601450 @jghoman This is an automated message from the Apache Git Service. To

[GitHub] [airflow] mik-laj merged pull request #6108: [AIRFLOW-5491] mark_tasks pydoc is incorrect

mik-laj merged pull request #6108: [AIRFLOW-5491] mark_tasks pydoc is incorrect URL: https://github.com/apache/airflow/pull/6108 This is an automated message from the Apache Git Service. To respond to the message, please log

  1   2   >