Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/19740
  
    I think that it is good to have top-level classes. I will check code later 
today.
    
    I feel that  `VectorBased...` is not clear. IMHO, `Vector` and `Column` is 
orthogonal. `Vector` can be used with `Row`.
    How about `ColumnBasedArray`/`ColumnBasedRow` or something? Other reviewers 
would find other names.


---

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

Reply via email to