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

SphinxQL INSERT > MaxPacketSize (0xffffff) broken since 2.8.0 #177

Closed
dennis-gr opened this issue Mar 20, 2019 · 5 comments
Closed

SphinxQL INSERT > MaxPacketSize (0xffffff) broken since 2.8.0 #177

dennis-gr opened this issue Mar 20, 2019 · 5 comments
Labels

Comments

@dennis-gr
Copy link

@dennis-gr dennis-gr commented Mar 20, 2019

INSERTing data bigger than 0xffffff via MySQL produces the following error since 2.8.0:

ERROR 1047 (08S01) at line 1: unknown command (code=99)

2.7.5 is working fine.

Changelog: fixed issue with large packet via SphinxQL interface

@githubmanticore
Copy link
Contributor

@githubmanticore githubmanticore commented Mar 20, 2019

➤ Stan commented:

Could you provide your config and INSERT into index SphinxQL statement as attached file to this issue?

@dennis-gr
Copy link
Author

@dennis-gr dennis-gr commented Mar 21, 2019

Sure, here you go: INSERT+Config.zip

Can be reproduced via mysql -P 9306 < HugeInsert.sql

@githubmanticore
Copy link
Contributor

@githubmanticore githubmanticore commented Mar 21, 2019

➤ Stan commented:

could you check that mysql_version_string = 5.5.21 fixes issue with large packet for you?

@dennis-gr
Copy link
Author

@dennis-gr dennis-gr commented Mar 21, 2019

No that doesn't change anything. This also wouldn't help with SphinxConnector.NET: I noticed this bug because I have an integration test for sending packets > 0xffffff which fails with 2.8.0 and up but works with previous versions and Sphinx.

@tomatolog
Copy link
Contributor

@tomatolog tomatolog commented Mar 22, 2019

I've just fixed issue with large packet via SphinxQL interface at
36df1a4 . You have to rebuild daemon to get issue fixed. That fix also will be released with next Manticore version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants