search

nijikokun / iConomy3

Please, if you fork iConomy only do so if you are really fixing or adding something useful!
7 stars 1 forks source link

MySQL Precision Crash #13

Closed Canisaur closed 13 years ago

Canisaur commented 13 years ago

When I start up my server using MySQL as the database, I get this error:

13:16:52 [SEVERE] [iConomy]: Could not create table for mysql: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Too big precision 254 specified for column 'balance'. Maximum is 65.

According to the MySQL docs, they've recently changed the Decimal precision from a max of 254 to a max of 65, because the MySQL math operations are only precise to 65 digits. (See http://dev.mysql.com/doc/refman/5.0/en/precision-math-decimal-changes.html)

Canisaur commented 13 years ago

Note that I'm currently using 3.0 still, as I'm waiting for DynamicMarket to update to 4.0. So if you've fixed this issue for 4.0, you can ignore this.

Canisaur commented 13 years ago

I found the issue in the source and fixed it. See my pull request :)

ShadowDrakken commented 13 years ago

why did you make 2 of the same issue?

also, I recommend using 4.1

Canisaur commented 13 years ago

Sorry, I didn't realize it automatically opens another 'issue' when you issue a pull request. I'll close this one.