Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/23781 )
Change subject: IMPALA-14284: Log the actual log files instead of symlinks in start-impala-cluster.py ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/23781/1/bin/start-impala-cluster.py File bin/start-impala-cluster.py: http://gerrit.cloudera.org:8080/#/c/23781/1/bin/start-impala-cluster.py@251 PS1, Line 251: def print_actual_log_file(log_symlink, timeout=5): > flake8: E302 expected 2 blank lines, found 1 Please fix. http://gerrit.cloudera.org:8080/#/c/23781/1/bin/start-impala-cluster.py@816 PS1, Line 816: print_actual_log_file(log_symlink) Shouldn't we print this before raising the RuntimeError too? It points to checking logs for details. -- To view, visit http://gerrit.cloudera.org:8080/23781 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id76c0a8bdfb221ab24ee315e2e273abca4257398 Gerrit-Change-Number: 23781 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Fri, 12 Dec 2025 17:10:45 +0000 Gerrit-HasComments: Yes
