mbk/pesterchum
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Close log file descriptors after write (low ulimit platform)

Browse source
This commit is contained in:
Dpeta 2022-03-23 18:32:59 +01:00
parent f092b8cb20
commit ee7a1fc043
1 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
profile.py
View file

@ -77,13 +77,23 @@ class PesterLog(object):
PchumLog.debug("post-error msg") PchumLog.debug("post-error msg")
self.convos[handle][format] = fp self.convos[handle][format] = fp
for (format, t) in modes.items(): for (format, t) in modes.items():
f = self.convos[handle][format] f = self.convos[handle][format]
if platform.system() == "Windows": f.write(t+"\r\n")
f.write(t+"\r\n")
else: # flush + fsync force a write, might not be required though.
f.write(t+"\r\n") for (format, t) in modes.items():
f.flush() f.flush()
os.fsync(f.fileno())
# This way the file descriptors are closed and reopened for every message,
# which is sub-optimal and definitely a performance drain but,
# otherwise we still run into the ulimit on platforms like MacOS fairly easily.
if ostools.isOSX() == True:
for (format, t) in modes.items():
self.finish(handle)
def finish(self, handle): def finish(self, handle):
if handle not in self.convos: if handle not in self.convos:
return return