wombatu-kun commented on code in PR #20385: URL: https://github.com/apache/hudi/pull/20385#discussion_r4305425654
########## hudi-ai-operator/skills/hudi-table-health/SKILL.md: ########## @@ -68,9 +68,12 @@ java -cp "$HUDI_UTILITIES_BUNDLE" org.apache.hudi.utilities.HoodieTableHealthChe --output JSON ``` -Always request `--output JSON`. Capture the exit code as well as stdout: `0` healthy, `1` unhealthy, -`2` the tool could not run (bad arguments, unreadable table, or a check that threw). On `2`, read -the `errors` array and stderr before saying anything about the table. +Always request `--output JSON`. Capture the exit code as well as stdout: `0` every check healthy or +skipped, `3` at least one check UNHEALTHY, `4` no check unhealthy but at least one could not run, +`2` the tool itself could not run (bad arguments or an unreadable table). There is no exit code `1`: Review Comment: A malformed command line (no arguments, a missing `--base-path`, an unknown flag) still exits `1`, because `HoodieTableHealthChecker.main` constructs `JCommander` outside its `try` and the `ParameterException` escapes `main`. Should `main` map it to `EXIT_ERROR` (fine as a follow-up), or should this paragraph say that `1` can also mean a rejected command line? -- 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]
