jscheffl commented on PR #54783: URL: https://github.com/apache/airflow/pull/54783#issuecomment-3217968803
@guan404ming / @Lee-W thanks for the discussion! My concerns are fixed by the rework and looks very good for me! > I probably need to check whether , exists in each option from the backend? That is really a limitation... that would be a + for having JSON support... mhm. I'd think it would be OK to assume no `,` is within the options and if somebody complaints then it can be addressed also in future. Would be really a nice case in my view. One option as resolution could be to offer to support JSON as well as plain text style... but might be un-needed complexity if nobody is using/needing it. Would make it easy for users to consume and still would leave the option for the nice cases... I would defer this to a future if really somebody needs it. -- 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]
