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

epugh commented on a change in pull request #288: URL: https://github.com/apache/solr/pull/288#discussion_r723637131 ## File path: solr/core/src/java/org/apache/solr/update/processor/DistributedZkUpdateProcessor.java ## @@ -314,6 +314,23 @@ public void processDelete(DeleteUpda

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

epugh commented on a change in pull request #288: URL: https://github.com/apache/solr/pull/288#discussion_r705673077 ## File path: solr/core/src/test/org/apache/solr/cloud/FullSolrCloudDistribCmdsTest.java ## @@ -298,6 +303,29 @@ public void testDeleteByIdCompositeRouterWithRo

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

epugh commented on a change in pull request #288: URL: https://github.com/apache/solr/pull/288#discussion_r705672767 ## File path: solr/core/src/java/org/apache/solr/update/processor/DistributedZkUpdateProcessor.java ## @@ -325,6 +325,24 @@ protected void doDeleteById(DeleteUp

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

epugh commented on a change in pull request #288: URL: https://github.com/apache/solr/pull/288#discussion_r704608615 ## File path: solr/core/src/java/org/apache/solr/update/processor/DistributedZkUpdateProcessor.java ## @@ -325,6 +325,24 @@ protected void doDeleteById(DeleteUp

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

epugh commented on a change in pull request #288: URL: https://github.com/apache/solr/pull/288#discussion_r704607164 ## File path: solr/core/src/test/org/apache/solr/cloud/FullSolrCloudDistribCmdsTest.java ## @@ -298,6 +303,29 @@ public void testDeleteByIdCompositeRouterWithRo