Skip to content

Patch to get_files to use mktemp

Zach Marshall requested to merge zmarshal/athena:master_get_files_mktemp into master

As suggested in ATEAM-858, patching get_files to use mktemp. Seems to be working in a few local tests, and seems to be a well-supported command. This avoids making use of /tmp in systems that might not have /tmp and might not have set $TMPDIR (e.g. might have set $TEMPDIR, or might use some other path for temporary files).

I recognize this is a delicate file to touch, so if there are any extra tests that anyone would like to suggest (including full tests on this MR), please feel free.

Merge request reports