Rename Class ending with Plugin to Module under modules dir

modules/aggs-matrix-stats/build.gradle

@@ -31,7 +31,7 @@ apply plugin: 'opensearch.yaml-rest-test'
 
 opensearchplugin {
   description 'Adds aggregations whose input are a list of numeric fields and output includes a matrix.'
-  classname 'org.opensearch.search.aggregations.matrix.MatrixAggregationPlugin'
+  classname 'org.opensearch.search.aggregations.matrix.MatrixAggregationModule'
   hasClientJar = true
 }
 

modules/aggs-matrix-stats/src/main/java/org/opensearch/search/aggregations/matrix/MatrixAggregationPlugin.java → modules/aggs-matrix-stats/src/main/java/org/opensearch/search/aggregations/matrix/MatrixAggregationModule.java

@@ -42,7 +42,7 @@
 
 import static java.util.Collections.singletonList;
 
-public class MatrixAggregationPlugin extends Plugin implements SearchPlugin {
+public class MatrixAggregationModule extends Plugin implements SearchPlugin {
     @Override
     public List<AggregationSpec> getAggregations() {
         return singletonList(

modules/aggs-matrix-stats/src/test/java/org/opensearch/search/aggregations/matrix/stats/InternalMatrixStatsTests.java

@@ -43,7 +43,7 @@
 import org.opensearch.search.aggregations.Aggregation;
 import org.opensearch.search.aggregations.InternalAggregation;
 import org.opensearch.search.aggregations.ParsedAggregation;
-import org.opensearch.search.aggregations.matrix.MatrixAggregationPlugin;
+import org.opensearch.search.aggregations.matrix.MatrixAggregationModule;
 import org.opensearch.search.aggregations.matrix.stats.InternalMatrixStats.Fields;
 import org.opensearch.search.aggregations.pipeline.PipelineAggregator.PipelineTree;
 import org.opensearch.test.InternalAggregationTestCase;
@@ -64,7 +64,7 @@ public class InternalMatrixStatsTests extends InternalAggregationTestCase<Intern
 
     @Override
     protected SearchPlugin registerPlugin() {
-        return new MatrixAggregationPlugin();
+        return new MatrixAggregationModule();
     }
 
     @Override

modules/aggs-matrix-stats/src/test/java/org/opensearch/search/aggregations/matrix/stats/MatrixStatsAggregatorTests.java

@@ -45,7 +45,7 @@
 import org.opensearch.index.mapper.NumberFieldMapper;
 import org.opensearch.plugins.SearchPlugin;
 import org.opensearch.search.aggregations.AggregatorTestCase;
-import org.opensearch.search.aggregations.matrix.MatrixAggregationPlugin;
+import org.opensearch.search.aggregations.matrix.MatrixAggregationModule;
 
 import java.util.Arrays;
 import java.util.Collections;
@@ -128,6 +128,6 @@ public void testTwoFields() throws Exception {
 
     @Override
     protected List<SearchPlugin> getSearchPlugins() {
-        return Collections.singletonList(new MatrixAggregationPlugin());
+        return Collections.singletonList(new MatrixAggregationModule());
     }
 }

modules/analysis-common/build.gradle

@@ -32,7 +32,7 @@ apply plugin: 'opensearch.internal-cluster-test'
 
 opensearchplugin {
   description 'Adds "built in" analyzers to OpenSearch.'
-  classname 'org.opensearch.analysis.common.CommonAnalysisPlugin'
+  classname 'org.opensearch.analysis.common.CommonAnalysisModule'
   extendedPlugins = ['lang-painless']
 }
 

modules/analysis-common/src/internalClusterTest/java/org/opensearch/analysis/common/QueryStringWithAnalyzersIT.java

@@ -48,7 +48,7 @@
 public class QueryStringWithAnalyzersIT extends OpenSearchIntegTestCase {
     @Override
     protected Collection<Class<? extends Plugin>> nodePlugins() {
-        return Arrays.asList(CommonAnalysisPlugin.class);
+        return Arrays.asList(CommonAnalysisModule.class);
     }
 
     /**

modules/analysis-common/src/main/java/org/opensearch/analysis/common/CommonAnalysisPlugin.java → modules/analysis-common/src/main/java/org/opensearch/analysis/common/CommonAnalysisModule.java

@@ -167,9 +167,9 @@
 
 import static org.opensearch.plugins.AnalysisPlugin.requiresAnalysisSettings;
 
-public class CommonAnalysisPlugin extends Plugin implements AnalysisPlugin, ScriptPlugin {
+public class CommonAnalysisModule extends Plugin implements AnalysisPlugin, ScriptPlugin {
 
-    private static final DeprecationLogger deprecationLogger = DeprecationLogger.getLogger(CommonAnalysisPlugin.class);
+    private static final DeprecationLogger deprecationLogger = DeprecationLogger.getLogger(CommonAnalysisModule.class);
 
     private final SetOnce<ScriptService> scriptService = new SetOnce<>();
 

modules/analysis-common/src/test/java/org/opensearch/analysis/common/ASCIIFoldingTokenFilterFactoryTests.java

@@ -51,7 +51,7 @@ public void testDefault() throws IOException {
                 .put(Environment.PATH_HOME_SETTING.getKey(), createTempDir().toString())
                 .put("index.analysis.filter.my_ascii_folding.type", "asciifolding")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_ascii_folding");
         String source = "Ansprüche";
@@ -68,7 +68,7 @@ public void testPreserveOriginal() throws IOException {
                 .put("index.analysis.filter.my_ascii_folding.type", "asciifolding")
                 .put("index.analysis.filter.my_ascii_folding.preserve_original", true)
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_ascii_folding");
         String source = "Ansprüche";

modules/analysis-common/src/test/java/org/opensearch/analysis/common/BaseWordDelimiterTokenFilterFactoryTestCase.java

@@ -60,7 +60,7 @@ public void testDefault() throws IOException {
                 .put(Environment.PATH_HOME_SETTING.getKey(), createTempDir().toString())
                 .put("index.analysis.filter.my_word_delimiter.type", type)
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";
@@ -78,7 +78,7 @@ public void testCatenateWords() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.catenate_words", "true")
                 .put("index.analysis.filter.my_word_delimiter.generate_word_parts", "false")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";
@@ -96,7 +96,7 @@ public void testCatenateNumbers() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.generate_number_parts", "false")
                 .put("index.analysis.filter.my_word_delimiter.catenate_numbers", "true")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";
@@ -115,7 +115,7 @@ public void testCatenateAll() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.generate_number_parts", "false")
                 .put("index.analysis.filter.my_word_delimiter.catenate_all", "true")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";
@@ -132,7 +132,7 @@ public void testSplitOnCaseChange() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.type", type)
                 .put("index.analysis.filter.my_word_delimiter.split_on_case_change", "false")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot";
@@ -149,7 +149,7 @@ public void testPreserveOriginal() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.type", type)
                 .put("index.analysis.filter.my_word_delimiter.preserve_original", "true")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";
@@ -186,7 +186,7 @@ public void testStemEnglishPossessive() throws IOException {
                 .put("index.analysis.filter.my_word_delimiter.type", type)
                 .put("index.analysis.filter.my_word_delimiter.stem_english_possessive", "false")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_word_delimiter");
         String source = "PowerShot 500-42 wi-fi wi-fi-4000 j2se O'Neil's";

modules/analysis-common/src/test/java/org/opensearch/analysis/common/CJKFilterFactoryTests.java

@@ -52,7 +52,7 @@ public class CJKFilterFactoryTests extends OpenSearchTokenStreamTestCase {
 
     @Before
     public void setup() throws IOException {
-        analysis = AnalysisTestsHelper.createTestAnalysisFromClassPath(createTempDir(), RESOURCE, new CommonAnalysisPlugin());
+        analysis = AnalysisTestsHelper.createTestAnalysisFromClassPath(createTempDir(), RESOURCE, new CommonAnalysisModule());
     }
 
     public void testDefault() throws IOException {

modules/analysis-common/src/test/java/org/opensearch/analysis/common/CommonAnalysisFactoryTests.java

@@ -50,7 +50,7 @@
 
 public class CommonAnalysisFactoryTests extends AnalysisFactoryTestCase {
     public CommonAnalysisFactoryTests() {
-        super(new CommonAnalysisPlugin());
+        super(new CommonAnalysisModule());
     }
 
     @Override

modules/analysis-common/src/test/java/org/opensearch/analysis/common/CommonGramsTokenFilterFactoryTests.java

@@ -58,7 +58,7 @@ public void testDefault() throws IOException {
             .build();
 
         try {
-            AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisPlugin());
+            AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisModule());
             Assert.fail("[common_words] or [common_words_path] is set");
         } catch (IllegalArgumentException e) {} catch (IOException e) {
             fail("expected IAE");
@@ -333,7 +333,7 @@ private Path createHome() throws IOException {
     }
 
     private static OpenSearchTestCase.TestAnalysis createTestAnalysisFromSettings(Settings settings) throws IOException {
-        return AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisPlugin());
+        return AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisModule());
     }
 
 }

modules/analysis-common/src/test/java/org/opensearch/analysis/common/CompoundAnalysisTests.java

@@ -102,8 +102,8 @@ private List<String> analyze(Settings settings, String analyzerName, String text
     }
 
     private AnalysisModule createAnalysisModule(Settings settings) throws IOException {
-        CommonAnalysisPlugin commonAnalysisPlugin = new CommonAnalysisPlugin();
-        return new AnalysisModule(TestEnvironment.newEnvironment(settings), Arrays.asList(commonAnalysisPlugin, new AnalysisPlugin() {
+        CommonAnalysisModule commonAnalysisModule = new CommonAnalysisModule();
+        return new AnalysisModule(TestEnvironment.newEnvironment(settings), Arrays.asList(commonAnalysisModule, new AnalysisPlugin() {
             @Override
             public Map<String, AnalysisProvider<TokenFilterFactory>> getTokenFilters() {
                 return singletonMap("myfilter", MyFilterTokenFilterFactory::new);

modules/analysis-common/src/test/java/org/opensearch/analysis/common/ConcatenateGraphTokenFilterFactoryTests.java

@@ -29,7 +29,7 @@ public class ConcatenateGraphTokenFilterFactoryTests extends OpenSearchTokenStre
     public void testSimpleTokenizerAndConcatenate() throws IOException {
         OpenSearchTestCase.TestAnalysis analysis = AnalysisTestsHelper.createTestAnalysisFromSettings(
             Settings.builder().put(Environment.PATH_HOME_SETTING.getKey(), createTempDir().toString()).build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("concatenate_graph");
@@ -47,7 +47,7 @@ public void testTokenizerCustomizedSeparator() throws IOException {
                 .put("index.analysis.filter.my_concatenate_graph.type", "concatenate_graph")
                 .put("index.analysis.filter.my_concatenate_graph.token_separator", "+")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_concatenate_graph");
@@ -65,7 +65,7 @@ public void testTokenizerEmptySeparator() throws IOException {
                 .put("index.analysis.filter.my_concatenate_graph.type", "concatenate_graph")
                 .put("index.analysis.filter.my_concatenate_graph.token_separator", "")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_concatenate_graph");
@@ -83,7 +83,7 @@ public void testPreservePositionIncrementsDefault() throws IOException {
                 .put("index.analysis.filter.my_concatenate_graph.type", "concatenate_graph")
                 .put("index.analysis.filter.my_concatenate_graph.token_separator", "+")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_concatenate_graph");
@@ -106,7 +106,7 @@ public void testPreservePositionIncrementsTrue() throws IOException {
                 .put("index.analysis.filter.my_concatenate_graph.token_separator", "+")
                 .put("index.analysis.filter.my_concatenate_graph.preserve_position_increments", "true")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_concatenate_graph");
@@ -132,7 +132,7 @@ public void testGraph() throws IOException {
                 .put("index.analysis.analyzer.my_analyzer.tokenizer", "whitespace")
                 .put("index.analysis.analyzer.my_analyzer.filter", "my_word_delimiter, my_concatenate_graph")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         String source = "PowerShot Is AweSome";
@@ -166,7 +166,7 @@ public void testInvalidSeparator() {
                     .put("index.analysis.filter.my_concatenate_graph.type", "concatenate_graph")
                     .put("index.analysis.filter.my_concatenate_graph.token_separator", "11")
                     .build(),
-                new CommonAnalysisPlugin()
+                new CommonAnalysisModule()
             )
         );
     }
@@ -187,7 +187,7 @@ public void testMaxGraphExpansion() throws IOException {
                 .put("index.analysis.analyzer.my_analyzer.tokenizer", "whitespace")
                 .put("index.analysis.analyzer.my_analyzer.filter", "my_word_delimiter, my_concatenate_graph")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
 
         String source = "PowerShot Is AweSome";

modules/analysis-common/src/test/java/org/opensearch/analysis/common/DisableGraphQueryTests.java

@@ -74,7 +74,7 @@ public class DisableGraphQueryTests extends OpenSearchSingleNodeTestCase {
 
     @Override
     protected Collection<Class<? extends Plugin>> getPlugins() {
-        return Collections.singleton(CommonAnalysisPlugin.class);
+        return Collections.singleton(CommonAnalysisModule.class);
     }
 
     @Before

modules/analysis-common/src/test/java/org/opensearch/analysis/common/EdgeNGramTokenFilterFactoryTests.java

@@ -52,7 +52,7 @@ public void testDefault() throws IOException {
                 .put(Environment.PATH_HOME_SETTING.getKey(), createTempDir().toString())
                 .put("index.analysis.filter.my_edge_ngram.type", "edge_ngram")
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_edge_ngram");
         String source = "foo";
@@ -69,7 +69,7 @@ public void testPreserveOriginal() throws IOException {
                 .put("index.analysis.filter.my_edge_ngram.type", "edge_ngram")
                 .put("index.analysis.filter.my_edge_ngram.preserve_original", true)
                 .build(),
-            new CommonAnalysisPlugin()
+            new CommonAnalysisModule()
         );
         TokenFilterFactory tokenFilter = analysis.tokenFilter.get("my_edge_ngram");
         String source = "foo";

modules/analysis-common/src/test/java/org/opensearch/analysis/common/EdgeNGramTokenizerTests.java

@@ -60,7 +60,7 @@ private IndexAnalyzers buildAnalyzers(Version version, String tokenizer) throws
             .put("index.analysis.analyzer.my_analyzer.tokenizer", tokenizer)
             .build();
         IndexSettings idxSettings = IndexSettingsModule.newIndexSettings("index", indexSettings);
-        return new AnalysisModule(TestEnvironment.newEnvironment(settings), Collections.singletonList(new CommonAnalysisPlugin()))
+        return new AnalysisModule(TestEnvironment.newEnvironment(settings), Collections.singletonList(new CommonAnalysisModule()))
             .getAnalysisRegistry()
             .build(idxSettings);
     }

modules/analysis-common/src/test/java/org/opensearch/analysis/common/ElisionFilterFactoryTests.java

@@ -49,7 +49,7 @@ public void testElisionFilterWithNoArticles() throws IOException {
 
         IllegalArgumentException e = expectThrows(
             IllegalArgumentException.class,
-            () -> AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisPlugin())
+            () -> AnalysisTestsHelper.createTestAnalysisFromSettings(settings, new CommonAnalysisModule())
         );
 
         assertEquals("elision filter requires [articles] or [articles_path] setting", e.getMessage());

modules/analysis-common/src/test/java/org/opensearch/analysis/common/HighlighterWithAnalyzersTests.java

@@ -65,7 +65,7 @@
 public class HighlighterWithAnalyzersTests extends OpenSearchIntegTestCase {
     @Override
     protected Collection<Class<? extends Plugin>> nodePlugins() {
-        return Arrays.asList(CommonAnalysisPlugin.class);
+        return Arrays.asList(CommonAnalysisModule.class);
     }
 
     public void testNgramHighlightingWithBrokenPositions() throws IOException {