vincbeck commented on PR #30300: URL: https://github.com/apache/airflow/pull/30300#issuecomment-1486971598
How this change would behave with queries already using double quotes? I know there had been many questions about this issue on Slack and the usual recommendation is to use double quotes. My guess is it wont work (I might be wrong). One solution would to detect if the string uses double quotes and if it does, no need to use `$$`. It would also be great to see this use case in the unit tests :) Besides that, thanks for trying to find a solution to this problem because, as mentioned, it had been brought up a lot on Slack -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
