rui-mo commented on code in PR #5090:
URL: https://github.com/apache/incubator-gluten/pull/5090#discussion_r1536970241
##########
gluten-ut/spark32/src/test/scala/io/glutenproject/utils/velox/VeloxTestSettings.scala:
##########
@@ -857,7 +857,6 @@ class VeloxTestSettings extends BackendTestSettings {
// decimal failed ut
.exclude("SPARK-34212 Parquet should read decimals correctly")
// Timestamp is read as INT96.
Review Comment:
This message was added when int64 type was unsupported for timestamp reader.
Could you update the message to illustrate why below tests are still disabled?
> .exclude("Migration from INT96 to TIMESTAMP_MICROS timestamp type")
.exclude("SPARK-10365 timestamp written and read as INT64 -
TIMESTAMP_MICROS")
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@gluten.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@gluten.apache.org
For additional commands, e-mail: commits-h...@gluten.apache.org
