From d41963c54697a3969a9abd1c96dfefc919faea8e Mon Sep 17 00:00:00 2001 From: Andrey K <32765888+akharitonov@users.noreply.github.com> Date: Sat, 24 Aug 2019 18:33:20 -0700 Subject: [PATCH] Fixed: missing brackets when appending proc info on OutOfMemory (#5530) * Fixed: missing brackets when appending proc info on OutOfMemory proc_stats.append was missing the set of brackets when adding a tuple to the list, which resulted in runtime error instead of correct Out of Memory message display. * Update memory_monitor.py --- python/ray/memory_monitor.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/python/ray/memory_monitor.py b/python/ray/memory_monitor.py index d74bb58b1..8feca07f5 100644 --- a/python/ray/memory_monitor.py +++ b/python/ray/memory_monitor.py @@ -43,7 +43,8 @@ class RayOutOfMemoryError(Exception): proc_stats = [] for pid in pids: proc = psutil.Process(pid) - proc_stats.append(get_rss(proc.memory_info()), pid, proc.cmdline()) + proc_stats.append((get_rss(proc.memory_info()), pid, + proc.cmdline())) proc_str = "PID\tMEM\tCOMMAND" for rss, pid, cmdline in sorted(proc_stats, reverse=True)[:10]: proc_str += "\n{}\t{}GiB\t{}".format(