ashokraminedi commented on code in PR #20130:
URL: https://github.com/apache/hudi/pull/20130#discussion_r4137805183


##########
hudi-spark-datasource/hudi-spark/src/test/scala/org/apache/spark/sql/hudi/common/TestVectorizedReadWithSchemaEvolution.scala:
##########
@@ -66,5 +66,44 @@ class TestVectorizedReadWithSchemaEvolution extends 
HoodieSparkSqlTestBase {
         }
       }
     }
+
+    test(s"Test INT to DECIMAL schema evolution with precision overflow for 
$tableType table") {
+      if (HoodieSparkUtils.isSpark3) {

Review Comment:
   Makes sense — since the vectorized reader is shared across the Spark 
profiles, the inherited Spark 3 guard isn't needed here.
   
   Updated in `889b09ac6c27` to remove the `isSpark3` guards from both tests, 
along with the now-unused `HoodieSparkUtils` import. This allows the tests to 
run on the Spark 4 profiles as well.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to