gopidesupavan commented on PR #62793:
URL: https://github.com/apache/airflow/pull/62793#issuecomment-3994494741
> LGTM with minor nits (already commented inline):
>
> 1. Missing `)` in f-string at line 267 of the operator — `({dialect_name}`
should be `({dialect_name})`
> 2. Docstring for `system_prompt` says "appended" but the behavior is
"replaces" — update the wording
> 3. Warning logs for PK/FK/index failures should include `table_name` for
context
thank you, if you would like to take a look one more time?
--
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]
