This is an automated email from the ASF dual-hosted git repository.
abhay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/master by this push:
new 42b8c973e RANGER-4193: ServiceTagsProcessor fails to handle update of
an existing Service-Resource
42b8c973e is described below
commit 42b8c973eb120f0dbf983d410d6ee888daa63ab8
Author: Abhay Kulkarni <[email protected]>
AuthorDate: Tue Apr 18 15:36:47 2023 -0700
RANGER-4193: ServiceTagsProcessor fails to handle update of an existing
Service-Resource
---
.../src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java
b/security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java
index fcbc31a99..f29304036 100644
---
a/security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java
+++
b/security-admin/src/main/java/org/apache/ranger/rest/ServiceTagsProcessor.java
@@ -154,7 +154,9 @@ public class ServiceTagsProcessor {
}
existing =
tagStore.getServiceResourceByGuid(resource.getGuid());
RangerPerfTracer.logAlways(perf);
- } else {
+ }
+
+ if (existing == null) {
if(MapUtils.isNotEmpty(resource.getResourceElements())) {
if(RangerPerfTracer.isPerfTraceEnabled(PERF_LOG_ADD_OR_UPDATE)) {
perf =
RangerPerfTracer.getPerfTracer(PERF_LOG_ADD_OR_UPDATE,
"tags.search_service_resource_by_signature(" + resourceId + ")");