Remove custom-codecs plugin (opensearch-project#9944)

This plugin has been moved to its own [repository][1]. [1]: https://github.com/opensearch-project/custom-codecs Signed-off-by: Andrew Ross <[email protected]>
andrross · Sep 8, 2023 · 5ad472f · 5ad472f
1 parent 4e1022e
commit 5ad472f
Show file tree

Hide file tree

Showing 24 changed files with 2 additions and 2,003 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -186,7 +186,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 - [Remote Store] Add support for Remote Translog Store stats in `_remotestore/stats/` API ([#9263](https://github.com/opensearch-project/OpenSearch/pull/9263))
 - Add support for query profiler with concurrent aggregation ([#9248](https://github.com/opensearch-project/OpenSearch/pull/9248))
 - Cleanup Unreferenced file on segment merge failure ([#9503](https://github.com/opensearch-project/OpenSearch/pull/9503))
-- Move ZStd to a plugin ([#9658](https://github.com/opensearch-project/OpenSearch/pull/9658))
+- Move zstd compression codec to external custom-codecs repository ([#9422](https://github.com/opensearch-project/OpenSearch/issues/9422])
 - [Remote Store] Add support for Remote Translog Store upload stats in `_nodes/stats/` API ([#8908](https://github.com/opensearch-project/OpenSearch/pull/8908))
 - Add instrumentation in rest and network layer. ([#9415](https://github.com/opensearch-project/OpenSearch/pull/9415))
 - Allow parameterization of tests with OpenSearchIntegTestCase.SuiteScopeTestCase annotation ([#9916](https://github.com/opensearch-project/OpenSearch/pull/9916))

diff --git a/modules/reindex/build.gradle b/modules/reindex/build.gradle
@@ -69,7 +69,6 @@ dependencies {
   testImplementation project(':modules:transport-netty4')
   // for parent/child testing
   testImplementation project(':modules:parent-join')
-  testImplementation project(':plugins:custom-codecs')
 }
 
 restResources {
@@ -96,5 +95,4 @@ forbiddenPatterns {
 tasks.named("bundlePlugin").configure {
   dependsOn("copyParentJoinMetadata")
   dependsOn("copyTransportNetty4Metadata")
-  dependsOn("copyCustomCodecsMetadata")
 }
diff --git a/.../reindex/src/internalClusterTest/java/org/opensearch/index/codec/MultiCodecReindexIT.java b/.../reindex/src/internalClusterTest/java/org/opensearch/index/codec/MultiCodecReindexIT.java
@@ -15,7 +15,6 @@
 import org.opensearch.action.support.ActiveShardCount;
 import org.opensearch.cluster.metadata.IndexMetadata;
 import org.opensearch.common.settings.Settings;
-import org.opensearch.index.codec.customcodecs.CustomCodecPlugin;
 import org.opensearch.index.engine.Segment;
 import org.opensearch.index.reindex.BulkByScrollResponse;
 import org.opensearch.index.reindex.ReindexAction;
@@ -47,7 +46,7 @@ public class MultiCodecReindexIT extends ReindexTestCase {
 
     @Override
     protected Collection<Class<? extends Plugin>> nodePlugins() {
-        return List.of(CustomCodecPlugin.class, ReindexModulePlugin.class);
+        return List.of(ReindexModulePlugin.class);
     }
 
     public void testReindexingMultipleCodecs() throws InterruptedException, ExecutionException {
@@ -57,10 +56,6 @@ public void testReindexingMultipleCodecs() throws InterruptedException, Executio
             "BEST_COMPRESSION",
             "zlib",
             "BEST_COMPRESSION",
-            "zstd_no_dict",
-            "ZSTD_NO_DICT",
-            "zstd",
-            "ZSTD",
             "default",
             "BEST_SPEED",
             "lz4",

diff --git a/plugins/custom-codecs/build.gradle b/plugins/custom-codecs/build.gradle
diff --git a/...decs/src/internalClusterTest/java/org/opensearch/index/codec/CodecCompressionLevelIT.java b/...decs/src/internalClusterTest/java/org/opensearch/index/codec/CodecCompressionLevelIT.java