chore: Remove TODO to align with main

himadripal · Jul 2, 2024 · 21b8ce4 · 21b8ce4
1 parent ae443d8
commit 21b8ce4
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
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,6 +115,7 @@ 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,8 +1396,6 @@ 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(