Merge pull request #5400 from atlanhq/fix-redis-wiring

MLH-1275 remove other redis implementations

Author: sriram-atlan

.github/workflows/maven.yml

@@ -31,7 +31,7 @@ on:
       - tagscanarymerge
       - fixlabels
       - interceptapis
-      - mlh-1240-custom-metadata-consistency
+      - fix-redis-wiring
 
 jobs:
   build:

common/src/main/java/org/apache/atlas/service/FeatureFlagStore.java

@@ -1,6 +1,5 @@
 package org.apache.atlas.service;
 
-import org.apache.atlas.service.redis.NoRedisServiceImpl;
 import org.apache.atlas.service.redis.RedisService;
 import org.apache.commons.lang.StringUtils;
 import org.springframework.beans.BeansException;
@@ -106,10 +105,6 @@ public void initialize() throws InterruptedException {
     private void validateDependencies() {
         LOG.info("Validating FeatureFlagStore dependencies...");
         try {
-            if (redisService instanceof NoRedisServiceImpl) {
-                return;
-            }
-
             // Test Redis connectivity with a simple operation
             String testKey = "ff:_health_check";
             redisService.putValue(testKey, "test");

common/src/main/java/org/apache/atlas/service/redis/NoRedisServiceImpl.java

@@ -1,7 +1,6 @@
 package org.apache.atlas.service.redis;
 
 import org.apache.atlas.AtlasException;
-import org.apache.atlas.annotation.ConditionalOnAtlasProperty;
 import org.apache.atlas.service.metrics.MetricUtils;
 import org.redisson.Redisson;
 import org.slf4j.Logger;
@@ -13,9 +12,8 @@
 import javax.annotation.PostConstruct;
 
 @Component
-@ConditionalOnAtlasProperty(property = "atlas.redis.service.impl")
 @Order(Ordered.HIGHEST_PRECEDENCE)
-public class RedisServiceImpl extends AbstractRedisService{
+public class RedisServiceImpl extends AbstractRedisService {
 
     private static final Logger LOG = LoggerFactory.getLogger(RedisServiceImpl.class);
     private static final long RETRY_DELAY_MS = 1000L;