Rename xref field in camelCase. Fix GNP-5564.
Linked to urgi-is/etl_script!54
Linked to urgi-is/database!16
Merge request reports
Activity
@jeremy.destin & @cyril.pommier a review xref changes is welcome
Resolved by Raphaël FloresIl faudra que j'y jette un oeil plus tard. Ca me rappel une campagne de reonmage qu'on avait fait coté data discovery il y a un an, mais dans l'autre sens (annotationId transformé en annotation_id) Il faudrait faire attentation à essayer d'être cohérent dune applis à l'autre, non ? ping @raphael.flores @celia.michotey
Edited by Cyril Pommier- Last reply by Cyril Pommier
added 6 commits
-
f94cc037...04ebf93c - 4 commits from branch
master - 58e7c9b9 - Rename xref field in camelCase. Fix GNP-5564.
- 702203b0 - Ahead on xref fields renaming. GNP-5564.
-
f94cc037...04ebf93c - 4 commits from branch
1 1 { 2 "transplant": { 2 "xref": { 3 3 "dynamic": "true", 4 4 "properties": { 5 "entry_type": { 5 "entryType": { 6 6 "type": "keyword" 7 7 }, 8 "linkedRessourcesID": { 8 "linkedResourcesID": { 9 9 "type": "keyword" 10 10 } -
Shouldn't
groupIdbe defined here too, as in the corresponding file in thedatabaserepositoty (https://forgemia.inra.fr/urgi-is/database/-/merge_requests/16/diffs#8f5bec7becd4667e8694741ece92546467e059e5) ? -
Well, it could be if this file was ever used. I can't find any reference to this file anywhere in the repository.
I wonder if it is not an artifact from @jeremy.destin's try to implement a Java indexer. Jeremy, do you know?
requested review from @raphael.flores
requested review from @jeremy.destin and removed review request for @raphael.flores
requested review from @raphael.flores and removed review request for @jeremy.destin
added 6 commits
-
702203b0...c5f705c0 - 4 commits from branch
master - 62d61ab6 - Rename xref field in camelCase. Fix GNP-5564.
- 969f5264 - Ahead on xref fields renaming. GNP-5564.
-
702203b0...c5f705c0 - 4 commits from branch