Miretpl commented on PR #68596:
URL: https://github.com/apache/airflow/pull/68596#issuecomment-5783693555

   One additional comment on the current implementation: it will also require 
all fields to be filled in top-level properties, e.g. `apiServer`, `workers`, 
.... For the e.g. `example` field, it seems a bit unnecessary to me at this 
moment, but on the other hand, I don't see a strong reason against it. If 
anyone looking at this PR will have some opinion on it, let me know.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to