ATLAS-5220: Removed unwanted comments

Author: VinayakMarraiya230920

distro/src/conf/atlas-application.properties

@@ -18,8 +18,6 @@
 
 #########  Graph Database Configs  #########
 
-# ATLAS-5220: Maven build-time backend selection (set via -Pstorage-* / -Pindex-* / -Pdist). Do not point runtime
-# config at a different storage/index backend unless you rebuild with matching profiles.
 atlas.build.storage.backend=${atlas.storage.backend}
 atlas.build.index.backend=${atlas.index.backend}
 

graphdb/janus/pom.xml

@@ -35,7 +35,6 @@
          Some dependencies, like slf4j are excluded from the jar because they are included in Atlas -->
 
     <properties>
-        <!-- Default storage backend for slim packaging; optional backends use storage-* profiles (ATLAS-5220). -->
         <atlas.storage.backend>hbase</atlas.storage.backend>
         <checkstyle.failOnViolation>true</checkstyle.failOnViolation>
         <checkstyle.skip>false</checkstyle.skip>
@@ -73,7 +72,6 @@
             <artifactId>commons-text</artifactId>
             <version>${commons-text.version}</version>
         </dependency>
-        <!-- Default index: Solr (+Lucene). Elasticsearch: -Pindex-elasticsearch on the WAR (ATLAS-5220). -->
         <dependency>
             <groupId>org.apache.lucene</groupId>
             <artifactId>lucene-analyzers-common</artifactId>
@@ -158,7 +156,6 @@
             <artifactId>janusgraph-server</artifactId>
             <version>${janusgraph.version}</version>
             <exclusions>
-                <!-- ATLAS-5220: optional Janus storage backends are profile-scoped; do not pull all from server. -->
                 <exclusion>
                     <groupId>com.codahale.metrics</groupId>
                     <artifactId>*</artifactId>
@@ -296,7 +293,6 @@
         </plugins>
     </build>
 
-    <!-- ATLAS-5220: optional storage backends. HBase stack only when atlas.storage.backend=hbase (default). -->
     <profiles>
         <profile>
             <id>atlas-storage-hbase</id>

graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphDatabase.java

@@ -78,9 +78,7 @@ public class AtlasJanusGraphDatabase implements GraphDatabase<AtlasJanusVertex,
     public static final String SOLR_ZOOKEEPER_URLS       = "atlas.graph.index.search.solr.zookeeper-urls";
     public static final String INDEX_BACKEND_LUCENE      = "lucene";
     public static final String INDEX_BACKEND_ES          = "elasticsearch";
-    /** Filtered at package time from Maven {@code atlas.storage.backend} (ATLAS-5220). */
     public static final String BUILD_STORAGE_BACKEND_CONF = "atlas.build.storage.backend";
-    /** Filtered at package time from Maven {@code atlas.index.backend} (ATLAS-5220). */
     public static final String BUILD_INDEX_BACKEND_CONF   = "atlas.build.index.backend";
     public static final String GRAPH_TX_LOG_CONF         = "tx.log-tx";
     public static final String GRAPH_TX_LOG_VERBOSE_CONF = "tx.recovery.verbose";
@@ -261,9 +259,6 @@ static JanusGraph initJanusGraph(Configuration config) {
         }
     }
 
-    /**
-     * Fail fast when runtime graph config does not match the Maven slim-build profile (ATLAS-5220).
-     */
     static void validateBuildBackendAlignment(Configuration janusConfig) {
         try {
             Configuration appConfig = ApplicationProperties.get();
@@ -442,7 +437,7 @@ public void cleanup() {
      * Register shortName -> FQCN in Janus {@code StandardStoreManager} without loading the class first.
      * Pre-loading with {@link Class#forName(String)} caused missing map entries when the class was not yet
      * visible to the caller's loader at static init time, leading Janus to treat {@code hbase2} as a class name.
-     * Implementation classes are loaded when the graph opens, as in the pre-ATLAS-5220 behavior.
+     * Implementation classes are loaded when the graph opens.
      */
     private static void safeRegisterStoreManager(String shortName, String managerClassName) {
         try {
@@ -498,9 +493,6 @@ private static void injectIndexProvider(String shortName, String providerClassNa
         LOG.debug("Registered Janus index provider {} -> {}", shortName, providerClassName);
     }
 
-    /**
-     * Register Janus shortName → implementation FQCN only when the class is loadable (ATLAS-5220 slim builds may omit JARs).
-     */
     private static void registerJanusOptionalBackends() {
         registerStoreIfLoadable("hbase2", HBASE2_STORE_MANAGER_CLASS);
         registerStoreIfLoadable("cql", CQL_STORE_MANAGER_CLASS);

pom.xml

@@ -86,7 +86,6 @@
         <antlr4.version>4.7</antlr4.version>
         <aopalliance.version>1.0</aopalliance.version>
         <aspectj.runtime.version>1.8.7</aspectj.runtime.version>
-        <!-- ATLAS-5220: atlas.audit.backend / atlas.storage.backend / atlas.index.backend for slim packaging; audit pairs with entity.repository.impl. -->
         <atlas.audit.backend>hbase</atlas.audit.backend>
         <atlas.index.backend>solr</atlas.index.backend>
         <atlas.storage.backend>hbase</atlas.storage.backend>
@@ -123,7 +122,6 @@
         <falcon.version>0.8</falcon.version>
         <fastutil.version>6.5.16</fastutil.version>
         <graph.index.backend>solr</graph.index.backend>
-        <!-- Align filtered test/distro graph backend with atlas.storage.backend default (ATLAS-5220). -->
         <graph.storage.backend>hbase2</graph.storage.backend>
         <gson.version>2.9.0</gson.version>
         <guava.version>25.1-jre</guava.version>
@@ -2292,8 +2290,6 @@
     </repositories>
 
     <profiles>
-        <!-- ATLAS-5220: combine with -Pstorage-* / -Paudit-* / -Pindex-*; put -Paudit-* after -Pdist.
-             Embedded: -Pdist,storage-hbase,audit-hbase,embedded-hbase-solr -->
         <profile>
             <id>dist</id>
             <activation>
@@ -2312,7 +2308,6 @@
                 <skipDocs>false</skipDocs>
             </properties>
         </profile>
-        <!-- ATLAS-5220: audit backend profiles set entity.repository.impl + atlas.audit.backend. -->
         <profile>
             <id>audit-cassandra</id>
             <activation>
@@ -2325,7 +2320,6 @@
                 <atlas.audit.backend>cassandra</atlas.audit.backend>
                 <atlas.war.audit.excludes>WEB-INF/lib/hbase-shaded-client-fixed-*.jar,</atlas.war.audit.excludes>
                 <entity.repository.impl>org.apache.atlas.repository.audit.CassandraBasedAuditRepository</entity.repository.impl>
-                <!-- distro filters ${entity.repository.properties} into atlas-application.properties; impl alone is not enough -->
                 <entity.repository.properties>atlas.EntityAuditRepository.impl=org.apache.atlas.repository.audit.CassandraBasedAuditRepository</entity.repository.properties>
             </properties>
         </profile>
@@ -2464,7 +2458,6 @@
                 <tests.solr.embedded>true</tests.solr.embedded>
             </properties>
         </profile>
-        <!-- ATLAS-5220: index backend (override with -Pindex-*). -->
         <profile>
             <id>index-elasticsearch</id>
             <activation>
@@ -2475,7 +2468,6 @@
             </activation>
             <properties>
                 <atlas.index.backend>elasticsearch</atlas.index.backend>
-                <!-- Filtered into atlas-application.properties as atlas.graph.index.search.backend -->
                 <graph.index.backend>elasticsearch</graph.index.backend>
             </properties>
         </profile>
@@ -2491,7 +2483,6 @@
                 <project.build.dashboardv2.gruntBuild>build</project.build.dashboardv2.gruntBuild>
             </properties>
         </profile>
-        <!-- ATLAS-5220: storage backend for atlas-graphdb-janus (override with -Pstorage-*). -->
         <profile>
             <id>storage-berkeleyje</id>
             <properties>

webapp/pom.xml

@@ -274,7 +274,6 @@
             </exclusions>
         </dependency>
 
-        <!-- ATLAS-5220: default dist/embedded builds use HBase; keep on WAR (profile-only deps were missing from WEB-INF/lib). -->
         <dependency>
             <groupId>org.apache.hadoop</groupId>
             <artifactId>hadoop-distcp</artifactId>
@@ -829,7 +828,6 @@
     </build>
 
     <profiles>
-        <!-- ATLAS-5220: graph storage JARs (profile id matches root -Pstorage-*). -->
         <profile>
             <id>storage-berkeleyje</id>
             <properties>