mortazakiani commented on PR #65758: URL: https://github.com/apache/airflow/pull/65758#issuecomment-4318562246
> I am not sure and following the bug ticket "why" thisis needed. In my view making this configurable adds un-needed complexity. > > I would dis-like to merge additional complexity if there is no described need. > > Side note: Plan mid-term is to remove gitsync completely from the chart and migrate to dag bundles. Adding new features to this inside the chart therefore is rather un-desired. > > @mortazakiani as you opened the bug ticket asking for the feature, is there any rationale or convincing argument why this complexity should be added? Thank you for reviewing my request. The point is that only one additional, longer path is being created for the DAGs, and this path is not actually necessary and can be removed. Although Git Sync is planned to be removed, it is simply an option -- 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]
