[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721078 ## File path: CONTRIBUTING.md ## @@ -348,17 +365,20 @@ they should give the

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721031 ## File path: CONTRIBUTING.md ## @@ -174,7 +184,14 @@ mkvirtualenv

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721141 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721179 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721024 ## File path: CONTRIBUTING.md ## @@ -174,7 +184,14 @@ mkvirtualenv

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721172 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721158 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721036 ## File path: CONTRIBUTING.md ## @@ -174,7 +184,14 @@ mkvirtualenv

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721107 ## File path: CONTRIBUTING.md ## @@ -430,6 +450,14 @@ See [Troubleshooting

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721073 ## File path: CONTRIBUTING.md ## @@ -348,17 +365,20 @@ they should give the

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721087 ## File path: CONTRIBUTING.md ## @@ -348,17 +365,20 @@ they should give the

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721013 ## File path: CONTRIBUTING.md ## @@ -142,6 +149,9 @@ environment and you

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721183 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721167 ## File path: CONTRIBUTING.md ## @@ -583,67 +611,84 @@ class

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721101 ## File path: CONTRIBUTING.md ## @@ -430,6 +450,14 @@ See [Troubleshooting

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721122 ## File path: CONTRIBUTING.md ## @@ -430,6 +450,14 @@ See [Troubleshooting

[GitHub] [airflow] dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on a change in pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#discussion_r312721116 ## File path: CONTRIBUTING.md ## @@ -430,6 +450,14 @@ See [Troubleshooting

[GitHub] [airflow] dimberman commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

dimberman commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#issuecomment-520195573 @potiuk dang that was a seriously productive flight. Looking over now :)

[jira] [Commented] (AIRFLOW-2923) LatestOnlyOperator cascade skip through all_done and dummy

[ https://issues.apache.org/jira/browse/AIRFLOW-2923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904526#comment-16904526 ] ASF GitHub Bot commented on AIRFLOW-2923: - m1racoli commented on pull request #5778:

[GitHub] [airflow] m1racoli opened a new pull request #5778: [AIRFLOW-2923][AIRFLOW-1784] Let operators inherit from BaseBranchOperator

m1racoli opened a new pull request #5778: [AIRFLOW-2923][AIRFLOW-1784] Let operators inherit from BaseBranchOperator URL: https://github.com/apache/airflow/pull/5778 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Assigned] (AIRFLOW-1784) SKIPPED status is being cascading wrongly

[ https://issues.apache.org/jira/browse/AIRFLOW-1784?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Cedrik Neumann reassigned AIRFLOW-1784: --- Assignee: Cedrik Neumann > SKIPPED status is being cascading wrongly >

[jira] [Assigned] (AIRFLOW-2923) LatestOnlyOperator cascade skip through all_done and dummy

[ https://issues.apache.org/jira/browse/AIRFLOW-2923?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Cedrik Neumann reassigned AIRFLOW-2923: --- Assignee: Cedrik Neumann > LatestOnlyOperator cascade skip through all_done and

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant fields. We

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520183227 I saw two remaining issues: (1) Serialized DAGs/Operators are big with many redundant

[GitHub] [airflow] coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520182466 > Pending Issues: > > * Issue displaying SubDags: This bug is because subdags are not

[GitHub] [airflow] coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon edited a comment on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520179192 > Pending Issues: > > * We still have the issue of `SerializedBaseOperator` being

[GitHub] [airflow] coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability

coufon commented on issue #5743: [AIRFLOW-5088][AIP-24] Persisting serialized DAG in DB for webserver scalability URL: https://github.com/apache/airflow/pull/5743#issuecomment-520179192 > Pending Issues: > > * We still have the issue of `SerializedBaseOperator` being displayed in

[GitHub] [airflow] ashb commented on issue #5461: [AIRFLOW-4835] Refactor render_template

ashb commented on issue #5461: [AIRFLOW-4835] Refactor render_template URL: https://github.com/apache/airflow/pull/5461#issuecomment-520175353 @bazph I took a week off, will look again in the morning This is an automated

[GitHub] [airflow] potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#issuecomment-520168998 All documentation and instructions are in

[GitHub] [airflow] potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#issuecomment-520167962 @nuclearpinguin ^^ also you can help to see if that works as expected :)

[GitHub] [airflow] potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

potiuk commented on issue #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777#issuecomment-520167375 Hey. While flying to NYC I managed to rebase, cleanup and trim-down pre-commit hook implementation for all our

[jira] [Commented] (AIRFLOW-5161) Add pre-commit hooks to run static checks for only changed files

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

[GitHub] [airflow] potiuk opened a new pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks

potiuk opened a new pull request #5777: [AIRFLOW-5161] Static checks are run automatically in pre-commit hooks URL: https://github.com/apache/airflow/pull/5777 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Resolved] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

[ https://issues.apache.org/jira/browse/AIRFLOW-5159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Potiuk resolved AIRFLOW-5159. --- Resolution: Fixed Fix Version/s: 1.10.5 > Optimise checklicence image build (do not

[jira] [Commented] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

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

[jira] [Commented] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

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

[jira] [Commented] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

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

[GitHub] [airflow] potiuk merged pull request #5774: [AIRFLOW-5159] Checklicence image is not built when not needed

potiuk merged pull request #5774: [AIRFLOW-5159] Checklicence image is not built when not needed URL: https://github.com/apache/airflow/pull/5774 This is an automated message from the Apache Git Service. To respond to the

[jira] [Created] (AIRFLOW-5161) Add pre-commit hooks to run static checks for only changed files

Jarek Potiuk created AIRFLOW-5161: - Summary: Add pre-commit hooks to run static checks for only changed files Key: AIRFLOW-5161 URL: https://issues.apache.org/jira/browse/AIRFLOW-5161 Project: Apache

[GitHub] [airflow] mik-laj opened a new pull request #5776: [AIRFLOW-XXX] Add section in TOC with all reference guides

mik-laj opened a new pull request #5776: [AIRFLOW-XXX] Add section in TOC with all reference guides URL: https://github.com/apache/airflow/pull/5776 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow

[GitHub] [airflow] potiuk commented on issue #5774: [AIRFLOW-5159] Checklicence image is not built when not needed

potiuk commented on issue #5774: [AIRFLOW-5159] Checklicence image is not built when not needed URL: https://github.com/apache/airflow/pull/5774#issuecomment-520145968 This one will be helpful to get pre-commits working (follow-up PR)

[jira] [Commented] (AIRFLOW-5160) Remove DAG count test

[ https://issues.apache.org/jira/browse/AIRFLOW-5160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16904409#comment-16904409 ] ASF GitHub Bot commented on AIRFLOW-5160: - BasPH commented on pull request #5775:

[GitHub] [airflow] BasPH opened a new pull request #5775: [AIRFLOW-5160] Remove example DAG count test

BasPH opened a new pull request #5775: [AIRFLOW-5160] Remove example DAG count test URL: https://github.com/apache/airflow/pull/5775 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[GitHub] [airflow] BasPH commented on issue #5461: [AIRFLOW-4835] Refactor render_template

BasPH commented on issue #5461: [AIRFLOW-4835] Refactor render_template URL: https://github.com/apache/airflow/pull/5461#issuecomment-520141652 @ashb any other remarks on this PR, or is it good to go? This is an automated

[jira] [Created] (AIRFLOW-5160) Remove DAG count test

Bas Harenslak created AIRFLOW-5160: -- Summary: Remove DAG count test Key: AIRFLOW-5160 URL: https://issues.apache.org/jira/browse/AIRFLOW-5160 Project: Apache Airflow Issue Type: Improvement

[jira] [Commented] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

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

[GitHub] [airflow] potiuk commented on issue #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users

potiuk commented on issue #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users URL: https://github.com/apache/airflow/pull/5773#issuecomment-520124304 Welcome! This is an automated message from the Apache Git

[GitHub] [airflow] potiuk commented on issue #5772: [AIRFLOW-XXX] Add iS2.co to list of airflow users

potiuk commented on issue #5772: [AIRFLOW-XXX] Add iS2.co to list of airflow users URL: https://github.com/apache/airflow/pull/5772#issuecomment-520124348 Welcome! This is an automated message from the Apache Git Service. To

[GitHub] [airflow] potiuk merged pull request #5772: [AIRFLOW-XXX] Add iS2.co to list of airflow users

potiuk merged pull request #5772: [AIRFLOW-XXX] Add iS2.co to list of airflow users URL: https://github.com/apache/airflow/pull/5772 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] potiuk merged pull request #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users

potiuk merged pull request #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users URL: https://github.com/apache/airflow/pull/5773 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow] potiuk opened a new pull request #5774: [AIRFLOW-5159] Checklicence image is not built when not needed

potiuk opened a new pull request #5774: [AIRFLOW-5159] Checklicence image is not built when not needed URL: https://github.com/apache/airflow/pull/5774 Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow

[jira] [Created] (AIRFLOW-5159) Optimise checklicence image build (do not build if not needed)

Jarek Potiuk created AIRFLOW-5159: - Summary: Optimise checklicence image build (do not build if not needed) Key: AIRFLOW-5159 URL: https://issues.apache.org/jira/browse/AIRFLOW-5159 Project: Apache

[GitHub] [airflow] bijanmoudi opened a new pull request #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users

bijanmoudi opened a new pull request #5773: [AIRFLOW-XXX] Adding AloPeyk to the list of Airflow users URL: https://github.com/apache/airflow/pull/5773 Make sure you have checked _all_ steps below. ### Jira - [ ] My PR addresses the following [Airflow