Skip to content

Commit

Permalink
Fix profile task type for local execution time.
Browse files Browse the repository at this point in the history
Instead of using REMOTE_PROCESS_TIME, it now uses `LOCAL_PROCESS_TIME`.

PiperOrigin-RevId: 567231250
Change-Id: I39f08bd41f4f6673784895a8002057a12da6b08c
  • Loading branch information
coeuvre authored and copybara-github committed Sep 21, 2023
1 parent 564ef7f commit 480b234
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -425,8 +425,7 @@ private SpawnResult start()
TerminationStatus terminationStatus;
try (SilentCloseable c =
Profiler.instance()
.profile(
ProfilerTask.REMOTE_PROCESS_TIME, spawn.getResourceOwner().getMnemonic())) {
.profile(ProfilerTask.LOCAL_PROCESS_TIME, spawn.getResourceOwner().getMnemonic())) {
needCleanup = true;
Subprocess subprocess = subprocessBuilder.start();
try {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ public enum ProfilerTask {
REMOTE_QUEUE("Remote execution queuing time", Threshold.FIFTY_MILLIS),
REMOTE_SETUP("Remote execution setup", Threshold.FIFTY_MILLIS),
FETCH("Remote execution file fetching", Threshold.FIFTY_MILLIS),
LOCAL_PROCESS_TIME("Local execution process wall time", Threshold.FIFTY_MILLIS),
VFS_STAT("VFS stat", Threshold.TEN_MILLIS, /* collectsSlowestInstances= */ true),
VFS_DIR("VFS readdir", Threshold.TEN_MILLIS, /* collectsSlowestInstances= */ true),
VFS_READLINK("VFS readlink", Threshold.TEN_MILLIS, /* collectsSlowestInstances= */ true),
Expand Down

0 comments on commit 480b234

Please sign in to comment.