jhrotko commented on code in PR #903: URL: https://github.com/apache/arrow-java/pull/903#discussion_r2510967933
########## vector/src/main/java/org/apache/arrow/vector/FixedSizeExtensionType.java: ########## Review Comment: This one is a bit tricky. For `ValueVectors` that are not an extension type and have a fixed width ex.: `FixedSizeBinaryVector.java`, they extend from `BaseFixedWidthVector` which is an abstract class with the method `getTypeWidth`. While `ExtensionTypeVectors` extend from `ExtensionTypeVector` hence cannot extend from the BaseFixedWidthVector` class. In some instances, for instance `UuidVector` it would be nice to have a way to calculate the (fixed) width of the vector directly with no major differences from `BaseFixedWidthVector`. This Interface was introduced to help solve this issue. In our case, we have other Extended vectors that do not have a fixed width. We could add this method to `ExtensionTypeVector` although it is not necessarily a fixed width -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
