Github user iyerr3 commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/195#discussion_r150638553
  
    --- Diff: src/ports/postgres/modules/utilities/validate_args.py_in ---
    @@ -262,6 +262,13 @@ def get_first_schema(table_name):
         return None
     # -------------------------------------------------------------------------
     
    +def drop_tables(table_list):
    +    """
    +        Drop tables specified in table_list.
    +    """
    +    drop_str = ', '.join([table for table in table_list])
    --- End diff --
    
    Why not just `', '.join(table_list)`?


---

Reply via email to