GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/20277
[SPARK-23090][SQL] polish ColumnVector ## What changes were proposed in this pull request? Several improvements: * make ColumnVector an interface instead of abstract class * provide a default implementation for the batch get methods * remove `arrayData`, as it's always the first child column vector * rename `getChildColumn` to `getChild`, which is more concise ## How was this patch tested? existing tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/cloud-fan/spark column-vector Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/20277.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #20277 ---- commit ba7ca0f168b35e381a9fd53ca59dd39d9dbd5920 Author: Wenchen Fan <wenchen@...> Date: 2018-01-16T16:38:28Z polish ColumnVector ---- --- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org