Removing esql depencies over ml

Author: afoucret

x-pack/plugin/esql/build.gradle

@@ -23,7 +23,7 @@ esplugin {
   name = 'x-pack-esql'
   description = 'The plugin that powers ESQL for Elasticsearch'
   classname = 'org.elasticsearch.xpack.esql.plugin.EsqlPlugin'
-  extendedPlugins = ['x-pack-esql-core', 'lang-painless', 'x-pack-ml']
+  extendedPlugins = ['x-pack-esql-core', 'lang-painless']
 }
 
 base {
@@ -34,7 +34,6 @@ dependencies {
   compileOnly project(path: xpackModule('core'))
   compileOnly project(':modules:lang-painless:spi')
   compileOnly project(xpackModule('esql-core'))
-  compileOnly project(xpackModule('ml'))
   implementation project(xpackModule('kql'))
   implementation project('compute')
   implementation project('compute:ann')
@@ -53,6 +52,7 @@ dependencies {
   testImplementation project(path: xpackModule('enrich'))
   testImplementation project(path: xpackModule('spatial'))
   testImplementation project(path: xpackModule('kql'))
+  testImplementation project(path: xpackModule('ml'))
 
   testImplementation project(path: ':modules:reindex')
   testImplementation project(path: ':modules:parent-join')

x-pack/plugin/esql/compute/build.gradle

@@ -12,7 +12,6 @@ dependencies {
   compileOnly project(':server')
   compileOnly project('ann')
   compileOnly project(xpackModule('core'))
-  compileOnly project(xpackModule('ml'))
   annotationProcessor project('gen')
   implementation 'com.carrotsearch:hppc:0.8.1'
 

x-pack/plugin/esql/compute/src/main/java/module-info.java

@@ -16,7 +16,6 @@
     // required due to dependency on org.elasticsearch.common.util.concurrent.AbstractAsyncTask
     requires org.apache.logging.log4j;
     requires org.elasticsearch.logging;
-    requires org.elasticsearch.ml;
     requires org.elasticsearch.tdigest;
     requires org.elasticsearch.geo;
     requires org.elasticsearch.xcore;

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/ChangePoint.java

@@ -19,7 +19,6 @@
 import org.elasticsearch.xpack.esql.core.type.DataType;
 import org.elasticsearch.xpack.esql.expression.NamedExpressions;
 import org.elasticsearch.xpack.esql.expression.Order;
-import org.elasticsearch.xpack.ml.aggs.changepoint.ChangePointDetectorImpl;
 
 import java.io.IOException;
 import java.util.List;
@@ -31,7 +30,7 @@
  * Plan that detects change points in a list of values. See also:
  * <ul>
  * <li> {@link org.elasticsearch.compute.operator.ChangePointOperator}
- * <li> {@link ChangePointDetectorImpl}
+ * <li> {@link org.elasticsearch.xpack.core.ml.aggs.changepoint.ChangePointDetector}
  * </ul>
  *
  * ChangePoint should always run on the coordinating node after the data is collected