curioustien commented on PR #45351: URL: https://github.com/apache/arrow/pull/45351#issuecomment-2660696942
@mapleFU @wgtmac I see that we already have a kinda similar flag called [arrow_extensions_enabled](https://github.com/curioustien/arrow/blame/parquet-decimal-test-feedback-2/cpp/src/parquet/properties.h#L916) from the JSON support. Should we just use this flag instead? It kinda makes sense to me to reuse this flag instead of introducing a new flag just for decimal type. I don't mind introducing a new flag like `smallest_decimal_enabled`, but I think it may make the API a little more complicated. -- 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]
