-
Notifications
You must be signed in to change notification settings - Fork 525
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
chore(pgwire): add simple 'unrecognized_param' error handle #3592
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3592 +/- ##
==========================================
- Coverage 74.38% 74.38% -0.01%
==========================================
Files 771 771
Lines 109013 109026 +13
==========================================
+ Hits 81089 81096 +7
- Misses 27924 27930 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
6734451
to
bb8de83
Compare
Hey @BowenXiao1999, this pull request failed to merge and has been dequeued from the merge train. If you believe your PR failed in the merge train because of a flaky test, requeue it by commenting with |
@Mergifyio requeue |
✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically |
5881baf
to
5f7c490
Compare
* chore(pgwire): add simple 'unrecognized_param' error handle * add some doc
…velabs#3592) * chore(pgwire): add simple 'unrecognized_param' error handle * add some doc
I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Detail see : #3404
Checklist
./risedev check
(or alias,./risedev c
)Refer to a related PR or issue link (optional)