You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by cloud-fan <gi...@git.apache.org> on 2018/01/02 13:40:10 UTC
[GitHub] spark pull request #20116: [SPARK-20960][SQL] make ColumnVector public
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20116#discussion_r159227467
--- Diff: sql/core/src/main/java/org/apache/spark/sql/sources/v2/vectorized/ColumnarBatch.java ---
@@ -87,19 +79,7 @@ public void remove() {
}
/**
- * Resets the batch for writing.
- */
- public void reset() {
- for (int i = 0; i < numCols(); ++i) {
- if (columns[i] instanceof WritableColumnVector) {
- ((WritableColumnVector) columns[i]).reset();
- }
- }
- this.numRows = 0;
--- End diff --
This doesn't matter. The `numRows` is only used when calling `rowsInterator`, and we always call `setNumRows` before calling `rowsIterator`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org