You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/04 14:01:34 UTC

[GitHub] [airflow] mik-laj edited a comment on pull request #13440: Additional properties should be allowed in provider schema

mik-laj edited a comment on pull request #13440:
URL: https://github.com/apache/airflow/pull/13440#issuecomment-753990787


   There are many more columns that are not used by Airflow but are only used by documentation. It seems to me that these will be all on this code block.
   https://github.com/apache/airflow/blob/3341d210cd265b51175d8f575e2ee73aa247f28b/airflow/provider.yaml.schema.json#L17-L180
   However, splitting this into several files seems to me to cause maintenance issues, but we can try to define one schema file that will contain all the information and have ``additionalProperties`` field set to `true` and a second specification that will be a subset of the first. This one can be used in runtime and can be forward-compatible.


----------------------------------------------------------------
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 the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org