bugraoz93 commented on PR #54034: URL: https://github.com/apache/airflow/pull/54034#issuecomment-3146127526
> Thanks for the PR. > > Airflow 2.x supported "" in extra parameter for connection and is still supported from the airflow cli. Since upcoming airflowctl would use APIs, I think problem would come if someone tries to import connections from older version of airflow, breaking compatibility. We might need to handle that. > > cc: @bugraoz93 @pierrejeambrun Thanks for flagging @shubhamraj-git! This could indeed be a problem and needs to be handled. This will highly likely reduce the migration effort too if we have backwards compatibility and allow users to migrate their connections to v3. **TLDR** `airflowctl` We can parse them to JSON before sending them to the API if an empty string is passed, to provide backwards compatibility in CTL. We should warn users that an empty string for extras would end up as JSON. I don't think this has to be included in the first release, as the airflow CLI will still support it. While deprecating duplicate functionalities, we have to implement it at least before deprecating the airflow CLI over airflowctl. It is also a small addition where we can add it. I will follow up with this -- 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]
