henry3260 commented on PR #68444: URL: https://github.com/apache/airflow/pull/68444#issuecomment-4700981279
> ~On second thought, we don't need this change at all. We can just redirect it with a warning. It will only create maintenance here with the additional code we are adding. Just adding a deprecation warning should be enough for this command, as Ash suggested.~ Edit: I missed the SELECT statement we deleted and replaced with airflowctl. To be honest, the GitHub UI tricked me the more I looked, as I pulled it locally to really see. My above comment applies 😅 I thought that local and what we are doing for actual parsing is the same. The deletion was hidden between additions https://github.com/apache/airflow/pull/68444/changes#diff-b28357c8748ec828697a10b2d785bffea845397916d410da18e0265b54155867L532-L536 This part indeed should be replaced with the airflowctl client as described in the issue and in AIP-94. What we need to ensure while deprecating, airflowctl should also have `--local` support before Airflow v4. Haha, no worries! The GitHub UI has definitely tricked me before too. 😅 -- 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]
