Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NEW Ensure commits are visible to seachers (fixes #274) #278

Merged
merged 1 commit into from May 16, 2022

Conversation

chillu
Copy link
Member

@chillu chillu commented Apr 7, 2020

This is required in situations where autoCommit.openSearcher is set to "false",
and you don't want to rely on core restarts to make committed index updates available to new searches.
It's already configured like this in the Common Web Platform environments,
so this change will mainly affect other environments (such as Silverstripe Platform).
To opt out of this new behaviour, set up a custom solrconfig.xml.

More details in #274

This is required in situations where autoCommit.openSearcher is set to "false",
and you don't want to rely on core restarts to make committed index updates available to new searches.
It's already configured like this in the Common Web Platform environments,
so this change will mainly affect other environments (such as Silverstripe Platform).
To opt out of this new behaviour, set up a custom solrconfig.xml.

More details in silverstripe#274
@chillu
Copy link
Member Author

chillu commented Apr 7, 2020

Sorry for the whitespace changes, I blame PHPStorm ;) Still testing this out on SilverStripe Platform, haven't successfully indexed content through it.

Also note that I haven't changed this for Solr 3, because who the hell cares about that.

@adrexia
Copy link
Contributor

adrexia commented Sep 8, 2021

Is there a reason this hasn't been merged yet?

@michalkleiner
Copy link
Contributor

I think everyone was waiting for @chillu as he indicated in his last comment he still hadn't successfully indexed content through it, so I think everyone was looking at this as a draft PR.

Can you please confirm what's the status of this @chillu? Thanks!

@emteknetnz
Copy link
Member

I've tested this on multiple platforms and things now work as expected

@emteknetnz emteknetnz merged commit 35c9484 into silverstripe:3 May 16, 2022
@emteknetnz
Copy link
Member

I've released this as 3.11.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants