pepijnve commented on PR #766:
URL: https://github.com/apache/arrow-java/pull/766#issuecomment-2904118948

   Just FYI, I'm experimenting a bit, and neither the `load` variant nor the 
boolean flag variant are particularly appealing.
   
   `load` feels a bit asymmetric. There's no need for any changes in the 
`export` methods as far as I can tell, so then you have three sets of methods: 
`export`, `import`, and `load`.
   
   The overload of `import` with a boolean flag is an option, but a bit ugly 
for `importVectorSchemaRoot(BufferAllocator,      ArrowArray, ArrowSchema, 
CDataDictionaryProvider)` where the behavior is controlled for both the array 
and the schema. Having two boolean flags seems even worser though.


-- 
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

Reply via email to