You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ab...@apache.org on 2023/04/18 22:37:04 UTC
[ranger] branch master updated: RANGER-4193: ServiceTagsProcessor fails to handle update of an existing Service-Resource
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 <ab...@apache.org>
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 + ")");