Feat integration directory - #1083
Conversation
| importer.linkRelativeToStructure(DefaultProfiles.RELATIVE_PROFILE_EXTERNAL_ID, getAcademyPrefix(), structureExternalId); | ||
| importer.linkRelativeToClass(DefaultProfiles.RELATIVE_PROFILE_EXTERNAL_ID, getAcademyPrefix(), structureExternalId); | ||
| } | ||
| return importer.getTransaction().commit().mapEmpty(); |
There was a problem hiding this comment.
Ça fait bizarre que dans la fonction preCommit on fasse appel à commit. Si pour une raison ou une autre on modifie la chaîne de handler d'aaf, je pense que ça aura des effets de bord.
De plus, quand avant on était atomique sur l'ensemble des structures là on commite par bout, ce qui fait qu'on peut finir dans un état incohérent.
There was a problem hiding this comment.
On commit déjà par bout car l'algo parse un fichier à la fois, pour un type de fichier, et les fichiers mélanges les élèves des structures.
There was a problem hiding this comment.
Pour le commit dans le precommit, je vais voir ce que je peux faire, de base il n'existe pas d'autre manière de découper pour eviter l'explosion de mémoire de neo4j. je vais voir si je trouve une idée
There was a problem hiding this comment.
J'ai poussé un refactoring qui ajoute une phase de finalization (postCommit), ce qui clarifie les frontières transactionnelles
|
8a68c23 to
1290c7c
Compare
|
3 participants
Description
Add integration on directory for AAF + optimize AAF. This pull request may have some impact has a part of optimisation to avoid load all the database neo4j in vertx or neo4j, by splitting in package and filter some queries on imported structure. The most important change must be made in configuration:
Fixes
https://edifice-community.atlassian.net/browse/ENABLING-1102?search_id=a65441d6-5e5f-4b47-92cc-917b2139cd8f
Type of change
Please check options that are relevant.
Which packages changed?
Please check the name of the package you changed
Tests
Reminder
Security flaws
Performance impacts (think bulk !)
Unit tests were replayed
Unit tests were added and/or changed
I have updated the reminder for the version including my modifications
All done ! 😃