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

indexer: better code #87

Merged
merged 1 commit into from
Aug 2, 2022
Merged

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented Jun 16, 2022

  • Eliminates exceptions for bulk indexing.
  • Better code with sourcery suggestions.

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

@rerowep rerowep self-assigned this Jun 16, 2022
@rerowep rerowep force-pushed the wep-better-bulk-index branch from 59c2d01 to 9b60c38 Compare June 16, 2022 19:48
* Eliminates exceptions for bulk indexing.
* Better code with sourcery suggestions.

Co-Authored-by: Peter Weber <peter.weber@rero.ch>
@rerowep rerowep force-pushed the wep-better-bulk-index branch from 9b60c38 to 41f5649 Compare June 16, 2022 21:19
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 72.793% when pulling 41f5649 on rerowep:wep-better-bulk-index into 2457349 on rero:staging.

@rerowep rerowep requested review from jma, BadrAly and Garfield-fr June 20, 2022 12:10
@rerowep rerowep merged commit 6114715 into rero:staging Aug 2, 2022
@rerowep rerowep deleted the wep-better-bulk-index branch August 31, 2022 12:23
# 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.

5 participants