HyukjinKwon opened a new pull request, #58602: URL: https://github.com/apache/spark/pull/58602
### What changes were proposed in this pull request? `SchemaConverters` tracked recursion depth per message-record name, so the `recursiveFieldMaxDepth` option could be exceeded by nesting through several different message types. Track the total depth reached along the current path so the limit applies across message types. The default (`recursiveFieldMaxDepth <= 0`) is unchanged. ### Why are the changes needed? Makes the existing `recursiveFieldMaxDepth` limit effective regardless of how the nesting is distributed across message types, rather than only per record name. ### Does this PR introduce _any_ user-facing change? No. It only affects schema conversion when `recursiveFieldMaxDepth` is already set to a positive value; the default behavior is unchanged. ### How was this patch tested? Existing protobuf recursion tests remain valid; the protobuf module compiles. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Isaac This pull request and its description were written by Isaac. Co-authored-by: Isaac <[email protected]> -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
