Github user kiszk commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19367#discussion_r141381707
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/vectorized/ColumnVectorSuite.scala
 ---
    @@ -181,9 +177,8 @@ class ColumnVectorSuite extends SparkFunSuite with 
BeforeAndAfterEach {
         assert(array.get(3, arrayType).asInstanceOf[ArrayData].toIntArray() 
=== Array(3, 4, 5))
       }
     
    -  test("struct") {
    -    val schema = new StructType().add("int", IntegerType).add("double", 
DoubleType)
    -    testVector = allocate(10, schema)
    +  val structType: StructType = new StructType().add("int", 
IntegerType).add("double", DoubleType)
    --- End diff --
    
    nit: Do we need `: StructType`?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to