diff options
| author | Robert Newson <rnewson@apache.org> | 2020-03-16 22:39:16 +0000 |
|---|---|---|
| committer | Joan Touzet <wohali@users.noreply.github.com> | 2020-04-27 17:06:01 -0400 |
| commit | e375b3d6944a20288b42cf3b94bcf88cc031b88d (patch) | |
| tree | 10ae243d16cfb93d948944fc307154fd2de1b78b | |
| parent | 70f94fe9a26e1739e77217b119e9b35e4dde5464 (diff) | |
| download | couchdb-e375b3d6944a20288b42cf3b94bcf88cc031b88d.tar.gz | |
View is partitioned if db and view are partitioned
We've seen a crash if DbPartitioned is false and ViewPartitioned is
true, which is obviously nonsense. The effect of the `nocase` is the
termination of the couch_index_server gen_server, which is a serious
amplification of a small (user-initiated) oddity.
| -rw-r--r-- | src/couch_mrview/src/couch_mrview_index.erl | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/src/couch_mrview/src/couch_mrview_index.erl b/src/couch_mrview/src/couch_mrview_index.erl index c96d87173..018060f83 100644 --- a/src/couch_mrview/src/couch_mrview_index.erl +++ b/src/couch_mrview/src/couch_mrview_index.erl @@ -252,16 +252,7 @@ set_partitioned(Db, State) -> DbPartitioned = couch_db:is_partitioned(Db), ViewPartitioned = couch_util:get_value( <<"partitioned">>, DesignOpts, DbPartitioned), - IsPartitioned = case {DbPartitioned, ViewPartitioned} of - {true, true} -> - true; - {true, false} -> - false; - {false, false} -> - false; - _ -> - throw({bad_request, <<"invalid partition option">>}) - end, + IsPartitioned = DbPartitioned andalso ViewPartitioned, State#mrst{partitioned = IsPartitioned}. |