garrensmith commented on a change in pull request #3481:
URL: https://github.com/apache/couchdb/pull/3481#discussion_r605101626



##########
File path: src/couch/src/couch_db.erl
##########
@@ -895,7 +895,7 @@ validate_doc_update(Db, Doc, GetDiskDocFun) ->
 
 validate_ddoc(Db, DDoc) ->
     try
-        ok = couch_index_server:validate(Db, couch_doc:with_ejson_body(DDoc))
+        ok = couch_mrview:validate(Db, couch_doc:with_ejson_body(DDoc))

Review comment:
       This isn't ideal. But it will be temporary. I want to remove 
couch_index_server but I want to do it in a few smaller PR's rather than one 
large one. This first step will get the validation_ddoc using couch_eval 
instead of couch_index_server and the old javascript proc system. 
   
   The next one will be fixes to the replication filtering and the vdu. A final 
one will be where I can remove couch_index_server and the fix this. 




-- 
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]


Reply via email to