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

2018-07-03 Thread asfgit
Github user asfgit closed the pull request at:

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


---


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

2018-07-01 Thread jingyimei
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 
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.




---