davisusanibar commented on code in PR #14389:
URL: https://github.com/apache/arrow/pull/14389#discussion_r1003824485
##########
java/vector/src/main/java/org/apache/arrow/vector/table/BaseTable.java:
##########
@@ -266,6 +270,48 @@ FieldVector getVector(int columnIndex) {
return fieldVectors.get(columnIndex);
}
+
+ /**
+ * Returns a copy of the vector with the given name, or throws
IllegalArgumentException if the name is not found.
+ * Names are case-sensitive.
+ *
+ * @param columnName The name of the vector to copy
+ * @return A copy of the Vector with the given name
+ * @throws IllegalArgumentException if the name is not the name of a vector
in the table.
+ */
+ public FieldVector getVectorCopy(String columnName) {
Review Comment:
> There's no abstract class as VSR doesn't inherit from anything. I have
considered adding an interface at some point that they both could implement to
make it easier to swap one for the other.
Ok, thank you.
> I'm not sure I understand this part of the comment.
This is related with the 1st question that initially VSR and Table are
growing independently and at some point they both could implement the same
contract.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
