ARROW-7085: Custom column builder for CSV reader#5880
ARROW-7085: Custom column builder for CSV reader#5880fexolm wants to merge 1 commit intoapache:masterfrom
Conversation
|
@fexolm The Parser, ColumnBuilder.. APIs are currently internal, and slightly messy. We'll have to think about cleaning them up before making them public. |
|
@pitrou the other option is to define a class inherited from ExtentionType with an additional method for creating column builder for that type. This solution wouldn't require changing stable API. |
|
No, this would have the same problem since |
|
@pitrou We would not mind if this ExtensionType based approach is implemented in internal namespace as well, so it will not appear as a stable API right away. |
|
Closing this PR since it has grown stale. I think it would be worth defining a public API for this, but we're going to have to take time to propose clean, well-documented interfaces rather than poking through internal APIs into the public API without any changes |
As discussed in ARROW-7085, I've implemented one of the ways to provide custom column builder to a column.
However, it doesn't look like final solution for me. Hope to hear your thought about possible implementation.