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

Improve logged message execute SQL commands #7156

Merged
merged 2 commits into from May 13, 2024
Merged

Conversation

fredzqm
Copy link
Contributor

@fredzqm fredzqm commented May 10, 2024

Description

Improves the SQL execute logging based on a feedback in the design doc.

Scenarios Tested

Sample Commands

Screenshot 2024-05-09 at 5 49 57 PM

@fredzqm fredzqm enabled auto-merge (squash) May 10, 2024 16:20
@fredzqm fredzqm changed the base branch from master to dataconnect May 10, 2024 18:00
@fredzqm fredzqm disabled auto-merge May 10, 2024 18:01
@joehan joehan merged commit cd961bc into dataconnect May 13, 2024
35 checks passed
@joehan joehan deleted the fz/execute-sql-polish branch May 13, 2024 16:57
joehan added a commit that referenced this pull request May 20, 2024
* Cloud SQL create and update polish (#7159)

* improve logged message execute SQL commands (#7156)

* Error handling and logging when linking CSQL instance (#7158)

* Display accurate message for INACCESSIBLE_SCHEMA error (#7157)

* handle INACCESSIBLE_SCHEMA case separately

* error msg

* m

* m

* m

* merge

* feedbacks

* Jh idx no metadata calls (#7179)

* Avoid triggering metadata server calls in idx

* Remove outdated service account check

* Update Firebase Logo (#7180)

* unleash turtles (#7174)

* unleash turtles

* Update CHANGELOG.md

* Flag flip for gen kit (#7175)

* Flag flip for gen kit

* Update CHANGELOG.md

* update icons

---------

Co-authored-by: Bryan Kendall <bkend@google.com>
Co-authored-by: joehan <joehanley@google.com>

* Use client instead of size 1 pool (#7182)

* Fix schema validation (#7185)

* Fix schema validation

* Also depend on redhat.yaml

* fix typo

---------

Co-authored-by: Fred Zhang <fredzqm@google.com>
Co-authored-by: Harold Shen <hlshen@google.com>
Co-authored-by: Bryan Kendall <bkend@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants