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 + ")");

Reply via email to