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

    https://github.com/apache/spark/pull/19601#discussion_r151311790
  
    --- Diff: 
sql/core/src/main/java/org/apache/spark/sql/execution/vectorized/ColumnVectorUtils.java
 ---
    @@ -93,28 +93,6 @@ public static void populate(WritableColumnVector col, 
InternalRow row, int field
         }
       }
     
    -  /**
    -   * Returns the array data as the java primitive array.
    -   * For example, an array of IntegerType will return an int[].
    -   * Throws exceptions for unhandled schemas.
    -   */
    -  public static Object toPrimitiveJavaArray(ColumnarArray array) {
    --- End diff --
    
    Why this method? Looks it is just used in test.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to