Github user kaknikhil commented on a diff in the pull request:
https://github.com/apache/madlib/pull/271#discussion_r191580503
--- Diff: src/madpack/utilities.py ---
@@ -33,6 +33,23 @@
this = os.path.basename(sys.argv[0]) # name of this script
+class AtomicFileOpen:
--- End diff --
do we need this `AtomicFileOpen`class ? I don't think it's used anywhere---
