XiaoMi / soar

SQL Optimizer And Rewriter
Apache License 2.0
8.67k stars 1.32k forks source link

关于COL.009判断逻辑的问题 #218

Open chrobin opened 5 years ago

chrobin commented 5 years ago

按照COL.009的规则描述,应该是建议使用DECIMAL,NUMERIC来替代FLOAT类型,但COL.009的判断逻辑中却因为字段中有mysql.TypeDecimal, mysql.TypeNewDecimal而触发,这样的判断和COL.009的描述是不是有冲突?

chrobin commented 5 years ago

https://github.com/XiaoMi/soar/blob/aeb7868114fac4fecdba162d24ef36867fc0909c/advisor/heuristic.go#L1622

martianzhang commented 5 years ago

COL.009 这个建议是从 《SQL 反模式》一书中学习而来,“尽可能不要使用浮点数。”