Skip to content

Feat integration directory - #1083

Open
vbillard91 wants to merge 4 commits into
devfrom
feat-integration-directory
Open

Feat integration directory#1083
vbillard91 wants to merge 4 commits into
devfrom
feat-integration-directory

Conversation

@vbillard91

@vbillard91 vbillard91 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

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:

  • check-exists-relationships must be turn to false. It create a gigantic cache in order to avoid some cheap queries in student creation process. The economy of such cache should be negligeable in the process (<20%) but the cost is huge on vertx or neo4j.

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.

  • Chore (PATCH)
  • Doc (PATCH)
  • Bug fix (PATCH)
  • New feature (MINOR)

Which packages changed?

Please check the name of the package you changed

  • admin
  • app-registry
  • archive
  • auth
  • cas
  • common
  • communication
  • conversation
  • directory
  • feeder
  • infra
  • portal
  • session
  • test
  • tests
  • timeline
  • workspace

Tests

  1. Describe here the tests you performed
  2. Step by step
  3. With expected results

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 ! 😃

@vbillard91
vbillard91 requested a review from juniorode July 17, 2026 12:30
Comment thread feeder/src/main/java/org/entcore/feeder/aaf1d/StudentImportProcessing1d2.java Outdated
Comment thread feeder/src/main/java/org/entcore/feeder/dictionary/structures/PostImport.java Outdated
@vbillard91
vbillard91 requested a review from dbreyton July 17, 2026 15:12
@sonarqubecloud

Copy link
Copy Markdown

@vbillard91
vbillard91 force-pushed the feat-integration-directory branch from 1290c7c to 00f7478 Compare September 3, 2026 13:28
@benjaminperez
benjaminperez force-pushed the dev branch 2 times, most recently from eed3cd5 to 2ec9bb0 Compare September 8, 2026 11:00
@juniorode
juniorode force-pushed the feat-integration-directory branch from 00f7478 to 830b2a6 Compare September 10, 2026 09:23
@sonarqubecloud

Copy link
Copy Markdown

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.

4 participants