Revert "chore: Remove TODO to align with main"

This reverts commit 21b8ce4.
himadripal · Jul 2, 2024 · fb6c81b · fb6c81b
1 parent c9f28e0
commit fb6c81b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/common/src/main/java/org/apache/comet/vector/CometPlainVector.java b/common/src/main/java/org/apache/comet/vector/CometPlainVector.java
@@ -115,7 +115,6 @@ public UTF8String getUTF8String(int rowId) {
       byte[] result = new byte[length];
       Platform.copyMemory(
           null, valueBufferAddress + offset, result, Platform.BYTE_ARRAY_OFFSET, length);
-
       if (!isUuid) {
         return UTF8String.fromBytes(result);
       } else {

diff --git a/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala b/spark/src/test/scala/org/apache/comet/exec/CometExecSuite.scala
@@ -1396,6 +1396,8 @@ class CometExecSuite extends CometTestBase {
   }
 
   test("RowToColumnar over RangeExec directly is eliminated for row output") {
+    // TODO: Temporarily ignore until we can determine why 3.4.3 plans differ from 3.4.2
+    assume(spark.version.equals("3.4.2"), "This test runs only on 3.4.2")
     Seq("true", "false").foreach(aqe => {
       Seq(500, 900).foreach { batchSize =>
         withSQLConf(