FANNG1 commented on PR #10106: URL: https://github.com/apache/gravitino/pull/10106#issuecomment-4004659955
Addressed the latest review feedback in commit 8bf02e1b0544bec56dace0dd0e2492a001765151. Updated items: - Align default to to match the built-in job behavior. - Reuse shared flat JSON parsing in and add on package-private static helpers. - Quote SQL identifiers consistently (including metadata table ) and update tests. - Fail fast for null partition values instead of writing string . - Add missing JavaDoc in . - Add clarifying comment in for Java 8 main / Java 17 tests split. - Stabilize IT warehouse config in . Note: I did not apply the log-redaction change in in this round. -- 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]
