You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@madlib.apache.org by jingyimei <gi...@git.apache.org> on 2018/07/02 01:02:08 UTC

[GitHub] madlib pull request #285: Madpack: fix install/reinstall not giving proper e...

GitHub user jingyimei opened a pull request:

    https://github.com/apache/madlib/pull/285

    Madpack: fix install/reinstall not giving proper error message

    Previously, uninstalling or reinstalling on a database that does not have MADlib already
    installed fails as expected. However, the info messages do not mention
    this failure and also keeps going to prepare database objects to
    install. This commit fixes this issue. If there is no madlib installed
    in database, madpack will show nothing found to uninstall/reinstall and
    stop.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/madlib/madlib bug_reinstall

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/madlib/pull/285.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #285
    
----
commit 677e76d202f620b71159367f1a1fd3aaab5513fc
Author: Jingyi Mei <jm...@...>
Date:   2018-07-02T00:49:24Z

    Madpack: fix install/reinstall not giving proper error message
    
    Previously, uninstalling or reinstalling on a database that does not have MADlib already
    installed fails as expected. However, the info messages do not mention
    this failure and also keeps going to prepare database objects to
    install. This commit fixes this issue. If there is no madlib installed
    in database, madpack will show nothing found to uninstall/reinstall and
    stop.

----


---

[GitHub] madlib issue #285: Madpack: fix install/reinstall not giving proper error me...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:

    https://github.com/apache/madlib/pull/285
  
    
    Refer to this link for build results (access rights to CI server needed): 
    https://builds.apache.org/job/madlib-pr-build/530/



---

[GitHub] madlib pull request #285: Madpack: fix install/reinstall not giving proper e...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/madlib/pull/285


---

[GitHub] madlib issue #285: Madpack: fix install/reinstall not giving proper error me...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:

    https://github.com/apache/madlib/pull/285
  
    
    Refer to this link for build results (access rights to CI server needed): 
    https://builds.apache.org/job/madlib-pr-build/529/



---