Skip to content

*: introduce FormatSQL and MustFormatSQL#22674

Closed
xhebox wants to merge 1 commit intopingcap:masterfrom
xhebox:injection_format
Closed

*: introduce FormatSQL and MustFormatSQL#22674
xhebox wants to merge 1 commit intopingcap:masterfrom
xhebox:injection_format

Conversation

@xhebox
Copy link
Contributor

@xhebox xhebox commented Feb 2, 2021

What problem does this PR solve?

Problem Summary: To target some flexible & dynamic SQL. It is extracted from #22644, one example case here

Check List

Tests

  • Unit test
  • Integration test

Release note

  • No release note

Signed-off-by: xhe <xw897002528@gmail.com>
@xhebox xhebox requested a review from a team as a code owner February 2, 2021 09:43
@xhebox xhebox requested review from wshwsh12 and removed request for a team February 2, 2021 09:43
@github-actions github-actions bot added the sig/execution SIG execution label Feb 2, 2021
@wshwsh12 wshwsh12 requested review from AilinKid and morgo February 5, 2021 08:35
@xhebox
Copy link
Contributor Author

xhebox commented Feb 19, 2021

It is one sub commit from #22644. It is expected to be merged after #22644, but #22644 got merged.

@xhebox xhebox closed this Feb 19, 2021
@xhebox xhebox deleted the injection_format branch February 20, 2021 05:55
@xhebox xhebox restored the injection_format branch February 23, 2021 02:27
@xhebox xhebox deleted the injection_format branch April 2, 2021 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/execution SIG execution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant