viirya commented on code in PR #40043:
URL: https://github.com/apache/arrow/pull/40043#discussion_r1486778895
##########
java/vector/src/main/java/org/apache/arrow/vector/FieldVector.java:
##########
@@ -60,6 +60,17 @@ public interface FieldVector extends ValueVector {
*/
List<ArrowBuf> getFieldBuffers();
+
+ /**
+ * Get the buffers for C Data Interface, (same size as getFieldVectors()
since it is their content).
+ * By default, it returns the same as getFieldBuffers().
+ *
+ * @return the buffers containing the data for this vector (ready for
exporting through C Data Interface)
+ */
+ default List<ArrowBuf> getCDataBuffers() {
+ return getFieldBuffers();
+ }
Review Comment:
I thought a few approaches to achieve the goal to fix this issue. This is
less intrusive one I can think of.
I've tried to modify `getFieldBuffers` directly. But it seems also used not
only for C Data Interface. So that change causes some test failures.
--
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
