FANNG1 opened a new pull request, #10370: URL: https://github.com/apache/gravitino/pull/10370
### What changes were proposed in this pull request? This PR updates the TMS quick-start verification command in `docs/table-maintenance-service/optimizer-quick-start.md`: - In step 7, use `log_dir` and grep both `output.log` and `error.log`. - Clarify that summary lines are typically in `output.log`, and may appear in `error.log` in some environments. ### Why are the changes needed? During end-to-end validation on Gravitino 1.2.0, rewrite summary lines such as: - `Rewritten data files: <N>` - `Added data files: <N>` - `Rewrite data files job completed successfully` were written to `output.log` instead of `error.log`. The previous doc command only checked `error.log`, which can cause false-negative verification. Fix: #N/A ### Does this PR introduce _any_ user-facing change? No API or property changes. Only documentation behavior/command guidance is corrected. ### How was this patch tested? Validated by running optimizer quick-start workflow end to end with: - Gravitino: `/Users/fanng/release/1.2.0/gravitino-1.2.0-bin` - Spark: `/Users/fanng/deploy/demo/spark-3.5.3-bin-hadoop3` Observed successful rewrite job and confirmed summary lines exist in: `/tmp/gravitino/jobs/staging/test/builtin-iceberg-rewrite-data-files/<job_id>/output.log`. -- 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]
