[VL] Fix timestamp precision loss in serializer

[zhli1142015]

What changes were proposed in this pull request?

By default PrestoVectorSerde use millisecond precision for timestamp data. We observed timestamp prescision loseing when doing broadcast serialization. We need to fix this by setting useLosslessTimestamp for PrestoVectorSerde.

import org.apache.spark.sql.functions.expr
import org.apache.spark.sql.catalyst.expressions._
spark.range(0, 5).toDF("id").createOrReplaceTempView("target")
spark.range(0, 10)
        .toDF("id")
        .withColumn("timestampOne", expr("timestamp_micros(id)"))
        .createOrReplaceTempView("source")

spark.sql("SELECT unix_micros(source.timestampOne) FROM source RIGHT OUTER JOIN target ON target.id = source.id").show(false)

Wrong result:

+-------------------------+
|unix_micros(timestampOne)|
+-------------------------+
|0                        |
|0                        |
|0                        |
|0                        |
|0                        |
+-------------------------+

Expected result:

+-------------------------+
|unix_micros(timestampOne)|
+-------------------------+
|0                        |
|1                        |
|2                        |
|3                        |
|4                        |
+-------------------------+

How was this patch tested?

UT

[github-actions]

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

• Other pull requests

[zhli1142015]

cc @PHILO-HE and @marin-ma , thanks.

[zhli1142015]

BTW, from this setting's comments, it's used for spilling also, not sure if this should be set to true for spilling also?
https://github.com/facebookincubator/velox/blob/main/velox/serializers/PrestoSerializer.h#L62

[marin-ma]

Thanks for this fix. Looks like it's already set to true in Velox Spill https://github.com/facebookincubator/velox/blob/b1252f26e112e9632e5a3e11df536387282c0259/velox/exec/SpillFile.cpp#L197-L198

https://github.com/facebookincubator/velox/blob/b1252f26e112e9632e5a3e11df536387282c0259/velox/exec/SpillFile.cpp#L23-L28

cc: @zhztheplayer

[zhli1142015]

Thanks for this fix. Looks like it's already set to true in Velox Spill https://github.com/facebookincubator/velox/blob/b1252f26e112e9632e5a3e11df536387282c0259/velox/exec/SpillFile.cpp#L197-L198

https://github.com/facebookincubator/velox/blob/b1252f26e112e9632e5a3e11df536387282c0259/velox/exec/SpillFile.cpp#L23-L28

cc: @zhztheplayer

Get it. Thanks for your info.

[PHILO-HE]

Looks good! cc @zhztheplayer