UmeshPatil-1 commented on code in PR #490:
URL: https://github.com/apache/atlas/pull/490#discussion_r2939464204
##########
repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasBusinessMetadataDefStoreV2.java:
##########
@@ -291,15 +319,35 @@ public AtlasVertex preDeleteByGuid(String guid) throws
AtlasBaseException {
throw new AtlasBaseException(AtlasErrorCode.TYPE_GUID_NOT_FOUND,
guid);
}
- if (existingDef != null) {
+ if (existingDef != null && !forceDelete) {
+ boolean hasIndexableAttribute = hasIndexableAttribute(existingDef);
+
+ if (!hasIndexableAttribute) {
Review Comment:
Deletion is low-frequency control-plane behavior. Registry-level caching for
this boolean adds invalidation complexity and hidden coherence risks for
marginal gain.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
