felipecrv commented on code in PR #35345:
URL: https://github.com/apache/arrow/pull/35345#discussion_r1344563962
##########
cpp/src/arrow/array/builder_nested.h:
##########
@@ -40,37 +40,46 @@ namespace arrow {
/// @{
// ----------------------------------------------------------------------
-// List builder
+// VarLengthListLikeBuilder
template <typename TYPE>
-class BaseListBuilder : public ArrayBuilder {
+class ARROW_EXPORT VarLengthListLikeBuilder : public ArrayBuilder {
Review Comment:
We don't necessarily need the `Base-` prefix IMO. The goal of this and
related interfaces is to allow virtual dispatching of actions that apply to all
the var-length list-like (excluding fixed-length, including map) arrays.
--
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]
