davisp commented on a change in pull request #2438: Reset a view shard if the
signature is wrong
URL: https://github.com/apache/couchdb/pull/2438#discussion_r365329706
##########
File path: src/couch_mrview/src/couch_mrview_index.erl
##########
@@ -127,6 +127,12 @@ open(Db, State0) ->
NewSt = couch_mrview_util:init_state(Db, Fd, State,
Header),
ensure_local_purge_doc(Db, NewSt),
{ok, NewSt};
+ {ok, {WrongSig, _}} ->
Review comment:
I'd rather not generalize here as I don't like that we're getting `{ok,
SomethingBad}` and I'd rather force our hand if it comes to that rather than
just resetting and papering over something likely serious. The headers are all
md5'ed so its got to be programmer error of some sort at that level.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
