-
Notifications
You must be signed in to change notification settings - Fork 25
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
indexer: fix bulk indexing #1202
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lauren-d
approved these changes
Sep 8, 2020
rerowep
force-pushed
the
wep-fix-bulk-index
branch
from
September 8, 2020 10:20
0983d80
to
f8d4ac2
Compare
rerowep
force-pushed
the
wep-fix-bulk-index
branch
6 times, most recently
from
September 8, 2020 12:14
e1d3b9e
to
6c381a4
Compare
iGormilhit
approved these changes
Sep 9, 2020
rerowep
force-pushed
the
wep-fix-bulk-index
branch
9 times, most recently
from
September 11, 2020 15:14
948c602
to
03e7104
Compare
rerowep
force-pushed
the
wep-fix-bulk-index
branch
3 times, most recently
from
September 15, 2020 07:05
4e63581
to
6890af5
Compare
Garfield-fr
approved these changes
Sep 23, 2020
rerowep
force-pushed
the
wep-fix-bulk-index
branch
from
September 23, 2020 11:58
6890af5
to
d20ee52
Compare
BadrAly
approved these changes
Sep 24, 2020
rerowep
force-pushed
the
wep-fix-bulk-index
branch
from
September 24, 2020 08:19
d20ee52
to
d4731f6
Compare
jma
approved these changes
Sep 24, 2020
lauren-d
approved these changes
Sep 24, 2020
rerowep
force-pushed
the
wep-fix-bulk-index
branch
from
September 25, 2020 09:53
d4731f6
to
020ef48
Compare
* Fixes race condition during parallel indexing of documents with persons creation. * Adds cli to post create persons from documents file. * Adds online update function for persons. without person : poetry run setup 503,53s user 35,71s system 62% cpu 14:19,69 total with person : poetry run setup 508,56s user 34,93s system 62% cpu 14:32,93 total with person enqueue: poetry run setup -k 495,98s user 34,25s system 62% cpu 14:12,54 total Co-Authored-by: Peter Weber <[email protected]>
rerowep
force-pushed
the
wep-fix-bulk-index
branch
from
September 29, 2020 08:08
020ef48
to
0ec59e8
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
persons creation.
Co-Authored-by: Peter Weber [email protected]
Why are you opening this PR?
Dependencies
My PR depends on the following
rero-ils-ui
's PR(s):How to test?
Code review check list