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