Skip to content

IULRDC-73: DataCORE - migrate solr 6 core to solr 9#242

Open
rdlebeau wants to merge 2 commits into
mainfrom
IULRDC-73_DataCORE_solr9
Open

IULRDC-73: DataCORE - migrate solr 6 core to solr 9#242
rdlebeau wants to merge 2 commits into
mainfrom
IULRDC-73_DataCORE_solr9

Conversation

@rdlebeau

@rdlebeau rdlebeau commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@rdlebeau rdlebeau requested a review from cjcolvar July 8, 2026 19:44

@cjcolvar cjcolvar left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good with respect to what I did in digital collections but I'm seeing errors in the CI build when it tries to create the solr core. From what I can tell it looks like it is having issues with the solr.LatLonType defined in the schema.xml. I don't see that field type being used in datacore as far as I can tell. If that's true then you can probably safely remove it and the _ll* fields that use it. I'm hopeful that once the schema.xml is setup correctly for solr 9 then these changes should work.

Is the solr reindex script coming in a different PR?

@rdlebeau rdlebeau force-pushed the IULRDC-73_DataCORE_solr9 branch from 26a9f97 to b2861c3 Compare July 9, 2026 13:47
@rdlebeau

rdlebeau commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

I have removed the unused solr.LatLonType and the _ll* fields that reference it from the schema.xml. This has resolved the errors in the solr part of the CI build.

@cjcolvar cjcolvar left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good for the solr changes as far as I can tell. I'll work on the reindex script in a separate PR.

@cjcolvar

Copy link
Copy Markdown

FWIW It looks like these config and schema files are already the ones in place on the solr 9 cores that were created before probably using the same files from digital collections.

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