Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19917 )

Change subject: IMPALA-12158: Added -f to "pgrep impalad" commands
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/19917/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19917/3//COMMIT_MSG@9
PS3, Line 9: IMPALA-10794 introduced names for threads, which messed up process 
names
I'd prefer all the process names were descriptive. Is there no way to fix that?


http://gerrit.cloudera.org:8080/#/c/19917/3//COMMIT_MSG@13
PS3, Line 13: with -f it will look into the program's path as well.
This could get false-positives if your development environment includes 
"impala" in a directory name.



--
To view, visit http://gerrit.cloudera.org:8080/19917
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I47e462a28d2f7c5ff202bf8614caafb57cc1e25b
Gerrit-Change-Number: 19917
Gerrit-PatchSet: 3
Gerrit-Owner: Gergely Fürnstáhl <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 24 May 2023 18:14:19 +0000
Gerrit-HasComments: Yes

Reply via email to