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

SOLR-17330: When not set, loadOnStartup defaults to true #2513

Merged
merged 4 commits into from
Jun 22, 2024

Conversation

psalagnac
Copy link
Contributor

SOLR-17330

Description

That's a pretty small change, replacing default value for loadOnStartup from false to true when the value is missing from core properties.
The default value when creating a new core is already true, so I don't expect any behavior change. Fixing this because I wasted some time understanding the code because of the inconsistent values.

I filled a Jira to track it, but I did not update the CHANGES file since no user should perceive the change (let me know if that's necessary).

Solution

Just set default to true.

Tests

None

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide

Copy link
Contributor

@epugh epugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm. Maybe a diggers variable name than tmp or even just inline it since we do that often? Love the cleanup

@epugh epugh self-assigned this Jun 19, 2024
@epugh
Copy link
Contributor

epugh commented Jun 19, 2024

Will merge tomorrow, assigned it to me so I do not forget!!

@epugh epugh merged commit 2193374 into apache:main Jun 22, 2024
3 checks passed
epugh added a commit that referenced this pull request Jun 22, 2024
@psalagnac psalagnac deleted the load-on-startup branch June 24, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants