Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19740#discussion_r150825051
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/vectorized/ColumnVectorSuite.scala
---
@@ -179,7 +179,7 @@ class ColumnVectorSuite extends SparkFunSuite with
BeforeAndAfterEach {
testVector.putArray(2, 3, 0)
testVector.putArray(3, 3, 3)
- val array = new ColumnVector.Array(testVector)
+ val array = new VectorBasedArray(testVector)
assert(array.get(0, arrayType).asInstanceOf[ArrayData].toIntArray()
=== Array(0))
--- End diff --
nit: Can we use `getArray()` instead of `get()`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
