Skip to content
New issue

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

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

Already on GitHub? # to your account

mef: reindex during entities reindexing #90

Merged
merged 1 commit into from
Aug 17, 2022
Merged

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented Jun 22, 2022

Co-Authored-by: Peter Weber peter.weber@rero.ch

@rerowep rerowep requested review from Garfield-fr, jma and BadrAly June 22, 2022 08:29
@rerowep rerowep self-assigned this Jun 22, 2022
@coveralls
Copy link

coveralls commented Jun 22, 2022

Coverage Status

Coverage decreased (-0.04%) to 72.72% when pulling a5b3a53 on wep-reindex-mef into b9ac078 on staging.

Copy link

@BadrAly BadrAly left a comment

Choose a reason for hiding this comment

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

during instead of durin in commit message

Copy link
Contributor

@jma jma left a comment

Choose a reason for hiding this comment

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

Please fix you commit message. Tests are missing.

@rerowep rerowep changed the title mef: reindex durin entities reindexing mef: reindex during entities reindexing Jul 28, 2022
@rerowep rerowep requested a review from jma July 28, 2022 12:49
@rerowep rerowep force-pushed the wep-reindex-mef branch 11 times, most recently from 900a541 to 791439c Compare August 3, 2022 08:52
@rerowep rerowep force-pushed the wep-reindex-mef branch 3 times, most recently from 228a8ce to 0596272 Compare August 11, 2022 13:22
@rerowep rerowep requested a review from jma August 11, 2022 14:27
@rerowep rerowep marked this pull request as draft August 11, 2022 14:38
@rerowep rerowep marked this pull request as ready for review August 16, 2022 12:15
* Reindex Mef record after entity indexing.
* Corrects function `get_all_pids_without_agents_and_viaf`.
* Deletes old constants `AGENTS`and `CONCEPTS`.
* Updates the dependencies lock file.

Co-Authored-by: Peter Weber <peter.weber@rero.ch>
@rerowep rerowep merged commit 4373288 into staging Aug 17, 2022
@rerowep rerowep deleted the wep-reindex-mef branch August 17, 2022 13:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants