adamreeve commented on code in PR #51239: URL: https://github.com/apache/arrow/pull/51239#discussion_r3963006851
########## cpp/src/parquet/properties.h: ########## @@ -68,6 +68,10 @@ constexpr int32_t kDefaultThriftStringSizeLimit = 100 * 1000 * 1000; // kDefaultStringSizeLimit. constexpr int32_t kDefaultThriftContainerSizeLimit = 1000 * 1000; +// Maximum schema nesting depth. This default value is conservatively small as +// some systems may not set a very large stack size. +constexpr int32_t kSchemaDepthLimit = 100; Review Comment: Nit: Should probably be called `kDefaultSchemaDepthLimit` for consistency with the other constants and to be clear it's a default that can be overridden. -- 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]
