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

    https://github.com/apache/madlib/pull/271#discussion_r192193755
  
    --- Diff: src/madpack/madpack.py ---
    @@ -824,6 +873,246 @@ def parse_arguments():
         # Get the arguments
         return parser.parse_args()
     
    +def run_install_check(args, testcase):
    +    schema = args['schema']
    --- End diff --
    
    This is install check code which is not refactored in this commit, and 
should be done as part of a different commit. We only moved `install-check` 
specific code from `main` to this function, since `main` was very huge.


---

Reply via email to