Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add fields to CLOUD_SQL_EXPORT_VALIDATION #23722

Closed
2 tasks done
ishiis opened this issue May 16, 2022 · 0 comments · Fixed by #23724
Closed
2 tasks done

Add fields to CLOUD_SQL_EXPORT_VALIDATION #23722

ishiis opened this issue May 16, 2022 · 0 comments · Fixed by #23724
Labels
area:providers kind:bug This is a clearly a bug

Comments

@ishiis
Copy link
Contributor

ishiis commented May 16, 2022

Apache Airflow Provider(s)

google

Versions of Apache Airflow Providers

apache-airflow-providers-google==5.0.0

Apache Airflow version

2.1.2

Operating System

GCP Container

Deployment

Composer

Deployment details

composer-1.17.1-airflow-2.1.2

What happened

I got a validation warning.

Same as #23613.

What you think should happen instead

The following fields are not implemented in CLOUD_SQL_EXPORT_VALIDATION.
The following fields should be added to CLOUD_SQL_EXPORT_VALIDATION.

  • sqlExportOptions
    • mysqlExportOptions
      • masterData
  • csvExportOptions
    • escapeCharacter
    • quoteCharacter
    • fieldsTerminatedBy
    • linesTerminatedBy

These are all the fields that have not been added.

https://cloud.google.com/sql/docs/mysql/admin-api/rest/v1beta4/operations#exportcontext

How to reproduce

No response

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers kind:bug This is a clearly a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant