madlib-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jingyimei <...@git.apache.org>
Subject [GitHub] madlib pull request #285: Madpack: fix install/reinstall not giving proper e...
Date Mon, 02 Jul 2018 01:02:08 GMT
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 <jmei@...>
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.

----


---

Mime
View raw message