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

feat:support oceanbase #15

Merged
merged 1 commit into from
Mar 3, 2025
Merged

feat:support oceanbase #15

merged 1 commit into from
Mar 3, 2025

Conversation

p1qt
Copy link

@p1qt p1qt commented Jan 9, 2025

Referring the mysql documentation and the golang mysql library, should be add use SERVER_MORE_RESULTS_EXISTS.
https://github.com/go-sql-driver/mysql
image

Signed-off-by: p1qt <1474789732@qq.com>

feat:support oceanbase

Signed-off-by: p1qt <1474789732@qq.com>
@p1qt
Copy link
Author

p1qt commented Jan 9, 2025

Fixes emqx/emqx#13866

@zmstone
Copy link
Member

zmstone commented Feb 4, 2025

Hi @p1qt
Thank you for the PR.
I had a closer look at the doc https://mariadb.com/kb/en/ok_packet/
it seems to indicate that there are more packets to follow.
It does not sound correct to me that right after authentication, oceanbase wants to send more data to the client.

@p1qt
Copy link
Author

p1qt commented Feb 24, 2025

🙂Thanks for your reply. We have not found any problems so far. Hope will be have a new better solution, because I'm also not good at MySQL.

@zmstone
Copy link
Member

zmstone commented Mar 3, 2025

Thank you @p1qt for the PR.
Will be fixed in EMQX 5.8.6.

@zmstone
Copy link
Member

zmstone commented Mar 11, 2025

Hi again @p1qt
It seems the correct mask should be SERVER_STATUS_NO_INDEX_USED but not SERVER_MORE_RESULTS_EXISTS ?

zmstone added a commit that referenced this pull request Mar 11, 2025
@zmstone zmstone mentioned this pull request Mar 11, 2025
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.

2 participants