kevingurney commented on issue #37978: URL: https://github.com/apache/arrow/issues/37978#issuecomment-1743747930
@rok - thank you for sharing this! My apologies - I didn't see this issue before. I just opened #37981 to add support for specifying a custom Array element delimiter. If I am not mistaken, it looks like https://github.com/apache/arrow/issues/30951 took the approach of trying to change the default delimiter to be `", "` instead of just `","`. My approach is slightly different in that it adds a new property to `arrow::PrettyPrintOptions` which allows clients to decide what they would like to use for the delimiter rather than trying to modify the default delimiter. If it seems OK, I will proceed with #37978 and #37981 for now to see if the community feels comfortable with this approach. If these changes end up getting merged, then it may make sense to mark https://github.com/apache/arrow/issues/30951 as closed since it kind of feels like #37981 partially address it (although, admittedly, it isn't exactly the same, since it doesn't change the default display to include a space after the comma). -- 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]
