[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-20 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/32#issuecomment-70786078 > BTW I've tried to use start_couch with [couch_mrview] extra app. It doesn't work in this case due to complex dependencies between couch_stats, couch_logger, couch

[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-20 Thread iilyak
Github user iilyak commented on the pull request: https://github.com/apache/couchdb-couch/pull/32#issuecomment-70725407 @kxepal BTW I've tried to use start_couch with [couch_mrview] extra app. It doesn't work in this case due to complex dependencies between `couch_stats`, `couch_logge

[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-20 Thread iilyak
Github user iilyak commented on the pull request: https://github.com/apache/couchdb-couch/pull/32#issuecomment-70682346 While working on test suite I had a need for start_couch(ExtraApps) and stop_couch(ExtraApps). I figure out that this API actually not side-effect free. Since when w

[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-20 Thread iilyak
Github user iilyak commented on the pull request: https://github.com/apache/couchdb-couch/pull/32#issuecomment-70678446 @kxepal [application:load/1 <> application:start/1](https://github.com/iilyak/couchdb-couch/blob/2552-unknown-metric-mrview-map_docs/src/test_util.erl#L144) --- If

[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-20 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/32#issuecomment-70641516 Yes, this fixes the issue. However, not sure about the fix correctness. `start_couch` is basically starts minimal CouchDB instance with the specified config

[GitHub] couchdb-couch pull request: Make sure we load all applications wit...

2015-01-19 Thread iilyak
GitHub user iilyak opened a pull request: https://github.com/apache/couchdb-couch/pull/32 Make sure we load all applications with stats defs couch_stats app relies on application:loaded_applications. While in the test suite we have to load the apps manually prior to start cou