Fix float inference to use DoubleParameter (64-bit) instead of FloatP… (#742)

* Fix float inference to use DoubleParameter (64-bit) instead of FloatParameter (32-bit)

Signed-off-by: Shubhambhusate <bhusates6@gmail.com>

* Add DoubleParameter with Primitive.DOUBLE to test_inference coverage

---------

Signed-off-by: Shubhambhusate <bhusates6@gmail.com>

Author: Shubhambhusate

src/databricks/sql/parameters/native.py

@@ -659,7 +659,7 @@ def dbsql_parameter_from_primitive(
     elif isinstance(value, str):
         return StringParameter(value=value, name=name)
     elif isinstance(value, float):
-        return FloatParameter(value=value, name=name)
+        return DoubleParameter(value=value, name=name)
     elif isinstance(value, datetime.datetime):
         return TimestampParameter(value=value, name=name)
     elif isinstance(value, datetime.date):

tests/unit/test_parameters.py

@@ -295,7 +295,8 @@ def test_tspark_param_ordinal(self):
             (BigIntegerParameter, Primitive.BIGINT),
             (BooleanParameter, Primitive.BOOL),
             (DateParameter, Primitive.DATE),
-            (FloatParameter, Primitive.FLOAT),
+            (DoubleParameter, Primitive.DOUBLE),
+            (DoubleParameter, Primitive.FLOAT),
             (VoidParameter, Primitive.NONE),
             (TimestampParameter, Primitive.TIMESTAMP),
             (MapParameter, Primitive.MAP),
@@ -305,7 +306,7 @@ def test_tspark_param_ordinal(self):
     def test_inference(self, _type: TDbsqlParameter, prim: Primitive):
         """This method only tests inferrable types.
 
-        Not tested are TinyIntParameter, SmallIntParameter DoubleParameter and TimestampNTZParameter
+        Not tested are TinyIntParameter, SmallIntParameter, FloatParameter and TimestampNTZParameter
         """
 
         inferred_type = dbsql_parameter_from_primitive(prim.value)