Fix ES 8.x compatibility

- Includes Jackson Core dependency
- Explicitly disable instrumentation on the plugins RestClient to
  avoid missing dependencies

Author: rvesse

README.md

@@ -48,15 +48,17 @@ If you are planning to use Elasticsearch with Docker, you should build a custom
 Dockerfile provided and use it instead
 
 ```
-docker build --tag=elasticsearch-telicent-plugin:7.17.14 .
+docker build --build-arg ELASTICSEARCH_VERSION=8.12.2 \
+  --build-arg PLUGIN_VERSION=8.12.2.1 \
+  --tag=elasticsearch-telicent-plugin:8.12.2.1 .
 ``` 
 
 This way the plugin will be preinstalled.
 
 Alternatively, you can use Telicent's Elasticsearch image
 
 ```
-docker pull telicent/elasticsearch
+docker pull telicent/elasticsearch:8.12.2.1
 ``` 
 
 ## Getting Started

pom.xml

@@ -5,17 +5,17 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>io.telicent.elasticsearch</groupId>
     <artifactId>SynonymsPlugin</artifactId>
-    <version>8.12.2.0</version>
+    <version>8.12.2.1</version>
     <description>A plugin for ElasticSearch providing an index-backed synonym
         handler
     </description>
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-		<elasticsearch.version>8.12.2</elasticsearch.version>
+	<elasticsearch.version>8.12.2</elasticsearch.version>
         <jdk.version>17</jdk.version>
         <testcontainer.version>1.19.2</testcontainer.version>
-        <jackson.core.version>2.16.1</jackson.core.version>
-        <jackson.databind.version>2.16.1</jackson.databind.version>
+        <jackson.core.version>2.17.0</jackson.core.version>
+        <jackson.databind.version>2.17.0</jackson.databind.version>
         <git-code-format-maven-plugin.version>4.3</git-code-format-maven-plugin.version>
     </properties>
     <build>
@@ -123,13 +123,11 @@
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-core</artifactId>
             <version>${jackson.core.version}</version>
-            <scope>provided</scope> <!-- Provided by Elasticsearch -->
         </dependency>
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-databind</artifactId>
             <version>${jackson.databind.version}</version>
-            <scope>compile</scope> <!-- Not provided by Elasticsearch -->
         </dependency>
         <dependency>
             <groupId>org.elasticsearch</groupId>

src/main/java/io/telicent/elasticsearch/IndexedSynonymParser.java

@@ -18,6 +18,7 @@
 import co.elastic.clients.elasticsearch.core.search.Hit;
 import co.elastic.clients.json.jackson.JacksonJsonpMapper;
 import co.elastic.clients.transport.ElasticsearchTransport;
+import co.elastic.clients.transport.instrumentation.NoopInstrumentation;
 import co.elastic.clients.transport.rest_client.RestClientTransport;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.node.ArrayNode;
@@ -144,7 +145,11 @@ public HttpAsyncClientBuilder customizeHttpClient(
 
         // Create the transport with a Jackson mapper
         final ElasticsearchTransport transport =
-                new RestClientTransport(builder.build(), new JacksonJsonpMapper());
+                new RestClientTransport(
+                        builder.build(),
+                        new JacksonJsonpMapper(),
+                        null,
+                        NoopInstrumentation.INSTANCE);
 
         // And create the API client
         final ElasticsearchClient client = new ElasticsearchClient(transport);