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


---

Reply via email to