featzhang commented on PR #27351: URL: https://github.com/apache/flink/pull/27351#issuecomment-4167059098
> @featzhang Thanks for the updates! I reviewed the latest changes, and it looks like the only remaining issue is the consistency between the documented error handling returning null behavior and the actual implementation described. > > cc @dylanhz if you have time, could you help take another look? Thanks for the review — all comments have been addressed in the latest force-push. Main updates include: * Improved `URL_DECODE_RECURSIVE` logic for clearer recursive termination. * Replaced hard-coded recursion control with validated `maxDepth`. * Ensured `maxDepth` validation is consistent with documented behavior. * Synced documentation examples and default-depth description with implementation. Please take another look when convenient. Thanks again! -- 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]
