-
Notifications
You must be signed in to change notification settings - Fork 10
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
PLT-573 | Add removeSupport in bulk API #2744
Conversation
...ry/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityGraphDiscoveryV2.java
Show resolved
Hide resolved
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.
Mostly LGTM. Left a minor clarification comment. Let's get 1 more approval before merge.
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.
Please check
- if we need to document the change , since we have priorities set for append and remove , the way API behaves.
- Error handling during remove , if relationship is not present.
- Also please squash all the commits to one commit
validate if attrName exists in appendRelationshipAttribute Evaluate attrName in appendRelationshipAttribute add util for appendRelationshipattribute util for processing appendRelationship attribute add utils for appendRelationshipAttribute add if condition to process appendRelationshipAttribute set appendRelationshipAttribute in context add condition to process appendRelationshipAttribute Add appendRelationshipAttribute util in EntityMutationContexr add flag to validate if appendRelationshipAttributes are visited Set flag if appendRelationshipAttribute is ever visited Set appendRelationshipAttribute Remove comments Add utils for removeRelationshipAttributes processRemoveRelationAttributes set removeRelationshipAttributes in context Add utils to process removeRelationshipAttributes and minor code refactoring code cleanup consume removeRelationshipAttributes add util to consumeRemoveRelationshipAttributes Update removeRelationshipAttribute util Expose method to fetch relationship
64125b1
to
71dbaeb
Compare
Squashed commits |
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.
Done
Change description
Type of change
Related issues
Checklists
Development
Security
Code review