Skip to content

Commit f67374b

Browse files
committed
Remove double subprocess communicate calls temporarily
Signed-off-by: Min Lim <min.yeol.lim@intel.com>
1 parent 17191c2 commit f67374b

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

gprofiler/utils/__init__.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -137,15 +137,15 @@ def start_process(
137137
**kwargs,
138138
)
139139

140-
def process_exit_handler() -> None:
141-
process.communicate()
142-
# Remove from _processes and do any cleanup
143-
cleanup_process_reference(process)
144-
logger.critical(f"Process {process.pid} exited, total processes tracked {len(_processes)}")
145-
146-
_processes.append(process)
147-
if direct_call:
148-
Thread(target=process_exit_handler, daemon=True).start()
140+
# def process_exit_handler() -> None:
141+
# process.communicate()
142+
# # Remove from _processes and do any cleanup
143+
# cleanup_process_reference(process)
144+
# logger.critical(f"Process {process.pid} exited, total processes tracked {len(_processes)}")
145+
#
146+
# _processes.append(process)
147+
# if direct_call:
148+
# Thread(target=process_exit_handler, daemon=True).start()
149149
return process
150150

151151

0 commit comments

Comments
 (0)