result convert to float automatically

[wjcnoob]

Introduction

return 24/48
output: 0

Contents

It would be great if the result could be automatically converted to float instead of adding toFloat() every time

Related work

[Shylock-Hg]

I don't think it's reasonable, not all users want float instead of integer in any scenes.

[wjcnoob]

But a indicator caculated like that is just wrong.

[wey-gu]

But an indicator calculated like that is just wrong.

/ in int stands for the mod(取模) and in float, it stands for division(除法).

You could check this in SQL:

SELECT 1/2,1.0/2

And the result will be:

You could play with it from here: https://sqliteonline.com/

[wjcnoob]

But an indicator calculated like that is just wrong.

/ in int stands for the mod(取模) and in float, it stands for division(除法).

You could check this in SQL:

SELECT 1/2,1.0/2

And the result will be:

You could play with it from here: https://sqliteonline.com/

Well, I'm using hive, and hive returns 0.5.
OK , it doesn't really matter.

[wey-gu]

Aha, thanks @wjcnoob for letting me know it's actually quite normal to treat / as division and another expression for the mod, which(like it was done in HIVE), I think is better than the current implicit approach in nGQL/SQL, but this behavior seems to be not possible to change due to it's correct(not the best way though) and will be a breaking change.

Could you please add toFloat() to one of the int values only(not as painful as toFloat() for all of them)? (sorry for this)

(root@nebula) [basketballplayer]> RETURN toFloat(24)/48
+------------------+
| (toFloat(24)/48) |
+------------------+
| 0.5              |
+------------------+
Got 1 rows (time spent 905/23873 us)

Thu, 03 Mar 2022 09:42:09 CST

Thanks again!!!

[wjcnoob]

Aha, thanks @wjcnoob for letting me know it's actually quite normal to treat / as division and another expression for the mod, which(like it was done in HIVE), I think is better than the current implicit approach in nGQL/SQL, but this behavior seems to be not possible to change due to it's correct(not the best way though) and will be a breaking change.

Could you please add toFloat() to one of the int values only(not as painful as toFloat() for all of them)? (sorry for this)

(root@nebula) [basketballplayer]> RETURN toFloat(24)/48
+------------------+
| (toFloat(24)/48) |
+------------------+
| 0.5              |
+------------------+
Got 1 rows (time spent 905/23873 us)

Thu, 03 Mar 2022 09:42:09 CST

Thanks again!!!

Of course, it's understandable

[wey-gu]

Thank you @wjcnoob !