domoritz commented on PR #39472: URL: https://github.com/apache/arrow/pull/39472#issuecomment-1883094253
> Based on the discussion here, do you still want me to test this fix as it stands or should I wait for another approach? I'm curious whether it works or not. > The internal factory functions that create builders will be creating instances of the non-extended class, but we could definitely use this strategy as a way to patch the prototypes w/ the static-property-initialized-by-an-IIFE style that defeats tree-shaking. How could that work? If someone imports only `makeBuilder` or `builderThroughIterable` for example, they might not import `Builder` and so we wouldn't execute the -- 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]
