Skip to content

Conversation

@alexisszmundy
Copy link
Contributor

@alexisszmundy alexisszmundy commented Jan 22, 2025

Will fix big files import

Copy link

Choose a reason for hiding this comment

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

Un peu étonné de voir du parsing xml dans le package controller mais c'est juste une opinion.

Par conte je pense que ça serait intéressant de basculer sur une dé-sérialisation basée sur un modèle objet, avec jackson par exemple ; ça fait écrire moins de code "bas niveau" sur les noeuds xml. Et même dans l'idéal avoir cette partie externalisée dans la lib Lunatic-Model

Copy link

Choose a reason for hiding this comment

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

Après discussion avec @loichenninger il y a bien nécessité d'écrire un peu de code plus bas niveau que juste de la désérialisation de tout le contenu d'un coup. On verra à l'occasion si on peut faire ça avec les fonctions de l'api jackson (➡️ voir si le changement est suffisamment simple à faire pour que ça vaille le coup) à la place de la "vielle" api javax.* (qui sera peut-être dépréciée dans une future version de java)

@loichenninger loichenninger merged commit b9113a8 into main Jan 31, 2025
4 checks passed
@loichenninger loichenninger deleted the devSequentialParserFix branch January 31, 2025 13:24
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.

5 participants