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

apply patch 12623 #132

Merged
merged 5 commits into from
Sep 18, 2023
Merged

apply patch 12623 #132

merged 5 commits into from
Sep 18, 2023

Conversation

pbibra
Copy link

@pbibra pbibra commented Sep 14, 2023

patch 12623: vitessio#12623

Note:
The regex in this PR is different from the one in the patch. The original author notes that there was a test failure with their regex change possibly which is the same test failure we hit here earlier. I'll put in a request upstream to make the same change, but I think this regex covers more cases.

Signed-off-by: Priya Bibra <[email protected]>
@pbibra pbibra requested a review from a team as a code owner September 14, 2023 22:11
Signed-off-by: Priya Bibra <[email protected]>
Signed-off-by: Priya Bibra <[email protected]>
Signed-off-by: Priya Bibra <[email protected]>
Copy link

@tanjinx tanjinx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pbibra pbibra merged commit 178f4cf into slack-vitess-r14.0.5 Sep 18, 2023
241 checks passed
@timvaillancourt timvaillancourt deleted the pbibra-patch-12623 branch May 27, 2024 18:09
timvaillancourt pushed a commit that referenced this pull request May 27, 2024
* apply patch 12623

Signed-off-by: Priya Bibra <[email protected]>

* fix unit test

Signed-off-by: Priya Bibra <[email protected]>

* revert regex

Signed-off-by: Priya Bibra <[email protected]>

* update regex

Signed-off-by: Priya Bibra <[email protected]>

---------

Signed-off-by: Priya Bibra <[email protected]>
timvaillancourt pushed a commit that referenced this pull request May 29, 2024
* apply patch 12623

Signed-off-by: Priya Bibra <[email protected]>

* fix unit test

Signed-off-by: Priya Bibra <[email protected]>

* revert regex

Signed-off-by: Priya Bibra <[email protected]>

* update regex

Signed-off-by: Priya Bibra <[email protected]>

---------

Signed-off-by: Priya Bibra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants