solr icon indicating copy to clipboard operation
solr copied to clipboard

SOLR-17614: cross-dc deletes can skip an HTTP hop

Open dsmiley opened this issue 1 year ago • 5 comments

https://issues.apache.org/jira/browse/SOLR-17614

Note: I ran the module's tests and they passed. But I have no experience with this module (or intention to use it) whatsoever. I'm just scratching an itch to remove a pointless HTTP connection and to use the super-cool EmbeddedSolrServer.

dsmiley avatar Jan 08 '25 03:01 dsmiley

Needs a CHANGES.txt; perhaps in Optimizations. "Cross-DC delete-by-query optimized to fetch deleted documents with one less HTTP hop per batch of IDs". I plan to merge it this weekend after adding that.

dsmiley avatar Feb 06 '25 19:02 dsmiley

This PR has had no activity for 60 days and is now labeled as stale. Any new activity will remove the stale label. To attract more reviewers, please tag people who might be familiar with the code area and/or notify the [email protected] mailing list. To exempt this PR from being marked as stale, make it a draft PR or add the label "exempt-stale". If left unattended, this PR will be closed after another 60 days of inactivity. Thank you for your contribution!

github-actions[bot] avatar Apr 08 '25 00:04 github-actions[bot]

Is there an issue that this is just querying a core instead of a collection after this change?

HoustonPutman avatar Apr 08 '25 00:04 HoustonPutman

Not that I know of.

But, I confess IMO Solr should know what is being addressed and treat that as the scope but Solr doesn't care -- in SolrCloud mode, the whole collection (the core is assigned to) is the scope, unless the magic param distrib=false is passed (for queries). Due to the spectre of back-compat... probably our best path forward is to look to the upcoming V2 API as the opportunity to rethink this.

dsmiley avatar Apr 08 '25 01:04 dsmiley

This PR has had no activity for 60 days and is now labeled as stale. Any new activity will remove the stale label. To attract more reviewers, please tag people who might be familiar with the code area and/or notify the [email protected] mailing list. To exempt this PR from being marked as stale, make it a draft PR or add the label "exempt-stale". If left unattended, this PR will be closed after another 60 days of inactivity. Thank you for your contribution!

github-actions[bot] avatar Jun 08 '25 00:06 github-actions[bot]

This PR is now closed due to 60 days of inactivity after being marked as stale. Re-opening this PR is still possible, in which case it will be marked as active again.

github-actions[bot] avatar Aug 09 '25 00:08 github-actions[bot]