-
Notifications
You must be signed in to change notification settings - Fork 32
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
In MySQL 8 median, stats_mode returns VARBINARY type #25
Comments
I'm having the same issue. I'm using the x64.dll for windows. |
Thanks for opening that up! I need to verify the type mismatch. @scanaximander About the .dll: Unfortunately, I don't have a Windows available to compile it. When the VARBINARY problem is fixed, it would be cool to find someone to compile a new dll. |
remove the row_number function fromm load.win.sql and load. mysql won't allow row_number since it is a keyword now. |
Yes, that's a short cut. But it is still in the dll and in the long run, I would like to have it clean. |
👍 |
@infusion thanks for the prompt reply. I'm not adept at compiling dlls, but when my workload clears up I'll give it a shot. |
I did not compile it previously, but the project files are included. Maybe it's enough to open it in Visual Studio, install the MySQL header files and done. |
@infusion Did you get a chance to verify the type mismatch? |
I looked into it from a users perspective:
It shows, when I put in a number, median returns not VARBINARY. How did you verify, the opposite? |
@infusion
|
Could you please test the type of the return value of median using the procedure with |
@infusion It is double. May be the issue occurs due to the connector jar. |
Is this problem new with MySQL 8? Do you get the correct number when you cast it? The question is, is this reproducible with other functions that might return real. |
ResultSetMetaData.getColumnTypeName() returns VARBINARY for median and stats_mode functions in MySQL version 8. In Previous versions it was DOUBLE. Have tested with two mysql connector jars. 5.1.44 and 8.0.11. Same happens in both. Let me know if you need more details.
udf_infusion.so.zip
The text was updated successfully, but these errors were encountered: