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
Feat/add validation to golden record process in pool #621
Feat/add validation to golden record process in pool #621
Conversation
1a712a8
to
42dbe24
Compare
5f146b9
to
a9c744b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just found a naming mistake
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
a9c744b
to
6363bc5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now besides the minor comments!
* Merge Maps with collections as values. | ||
* The collections with the same key in the different maps are concatenated | ||
*/ | ||
fun <KEY, VALUE> mergeMapsWithCollectionInValue( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe name it mergeMultiMaps
?
https://en.wikipedia.org/wiki/Multimap
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The core of the functionality is that the collections in the value part of the map are concatenated.
This should be expressed in the name.
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/BusinessPartnerBuildService.kt
Outdated
Show resolved
Hide resolved
...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/TaskStepFetchAndReserveService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
bpdm-pool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/RequestValidationService.kt
Outdated
Show resolved
Hide resolved
6363bc5
to
16e342d
Compare
@martinfkaeser from your prerspective are there still open points that need to be addressed in this pull request? |
Description
Upsert Business Partners from Cleaning Result - implement validation for legel entities, sites and addresses
add validation to the golden record process
fix validation of address identifiers
Tests are moved to the next step
Solves: #556