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

fix: sql-formatter #1152

Merged
merged 3 commits into from
Feb 8, 2023
Merged

fix: sql-formatter #1152

merged 3 commits into from
Feb 8, 2023

Conversation

jczhong84
Copy link
Collaborator

@jczhong84 jczhong84 commented Feb 8, 2023

sql-formatter is not working properly.
and also the behavior is not consistent between adhoc and datadoc

cause:
https://github.com/pinterest/querybook/pull/1141/files#diff-f96c7f939d2f6190281fe1288630f6ce6e7efec269cd71a04a16a6379686e38fR185

fix

  • make the silent option to be default false for both adhoc and datadoc
  • format either fail or succeed on the whole query, not per statement

@jczhong84 jczhong84 changed the title fix: sql-formmater fix: sql-formatter Feb 8, 2023
czgu
czgu previously approved these changes Feb 8, 2023
@jczhong84 jczhong84 merged commit efdbd7b into pinterest:master Feb 8, 2023
@jczhong84 jczhong84 deleted the fix/formatter branch February 23, 2023 01:58
rohan-sh1 pushed a commit to CAI-TECHNOLOGIES/cai-ext-db-explorer that referenced this pull request Apr 11, 2023
* fix: sql-formmater

* remove originalStatementText

* try catch on whole query
aidenprice pushed a commit to arrowtail-precision/querybook that referenced this pull request Jan 3, 2024
* fix: sql-formmater

* remove originalStatementText

* try catch on whole query
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