[GitHub] [solr] makosten commented on pull request #288: SOLR-15705: A deleteById request without _route_ param for implicit router could be sent to all shard leaders

[GitBox]

makosten commented on pull request #288:
URL: https://github.com/apache/solr/pull/288#issuecomment-947905418


   Here is a process flow of a DeleteByQuery. The doDistribDeleteByQuery is 
where a leader forwards the command to its replicas. The doDeleteByQuery in 
DistributedZkUpdateProcessor is where the command is forwarded to the shard 
leaders, so I do think that my logic of forwarding the request to 
doDeleteByQuery is in the right location, although the whole approach is not 
optimal. I did try modeling how DeleteByQuery works by forwarding the request 
to shard leaders, but found that the local deleteById, actually deleting the 
document from the index, does not happen if the route field value is missing.
   
   
![image](https://user-images.githubusercontent.com/19311245/138144905-cf7a0047-1490-46a3-8c8e-02958b142af7.png)
   


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

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org