update analytics job creation to use _explain api endpoint for modelM…

…emoryLimit estimate
elastic · Nov 25, 2019 · 758661b · 758661b
1 parent adc11a5
commit 758661b
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/...ics/pages/analytics_management/components/create_analytics_form/create_analytics_form.tsx b/...ics/pages/analytics_management/components/create_analytics_form/create_analytics_form.tsx
@@ -118,7 +118,8 @@ export const CreateAnalyticsForm: FC<CreateAnalyticsFormProps> = ({ actions, sta
  delete jobConfig.model_memory_limit;
  const resp = await ml.dataFrameAnalytics.estimateDataFrameAnalyticsMemoryUsage(jobConfig);
  setFormState({
- modelMemoryLimit: resp.expected_memory_without_disk,
+ modelMemoryLimit:
+ resp.memory_estimation && resp.memory_estimation.expected_memory_without_disk,
  });
  } catch (e) {
  setFormState({

diff --git a/x-pack/legacy/plugins/ml/public/application/services/ml_api_service/data_frame_analytics.js b/x-pack/legacy/plugins/ml/public/application/services/ml_api_service/data_frame_analytics.js
@@ -49,7 +49,7 @@ export const dataFrameAnalytics = {
  },
  estimateDataFrameAnalyticsMemoryUsage(jobConfig) {
  return http({
- url: `${basePath}/data_frame/analytics/_estimate_memory_usage`,
+ url: `${basePath}/data_frame/analytics/_explain`,
  method: 'POST',
  data: jobConfig
  });

diff --git a/x-pack/legacy/plugins/ml/server/client/elasticsearch_ml.js b/x-pack/legacy/plugins/ml/server/client/elasticsearch_ml.js
@@ -173,7 +173,7 @@ export const elasticsearchJsPlugin = (Client, config, components) => {
  ml.estimateDataFrameAnalyticsMemoryUsage = ca({
  urls: [
  {
- fmt: '/_ml/data_frame/analytics/_estimate_memory_usage',
+ fmt: '/_ml/data_frame/analytics/_explain',
  }
  ],
  needBody: true,

diff --git a/x-pack/legacy/plugins/ml/server/routes/data_frame_analytics.js b/x-pack/legacy/plugins/ml/server/routes/data_frame_analytics.js
@@ -93,7 +93,7 @@ export function dataFrameAnalyticsRoutes({ commonRouteConfig, elasticsearchPlugi
 
  route({
  method: 'POST',
- path: '/api/ml/data_frame/analytics/_estimate_memory_usage',
+ path: '/api/ml/data_frame/analytics/_explain',
  handler(request) {
  const callWithRequest = callWithRequestFactory(elasticsearchPlugin, request);
  return callWithRequest('ml.estimateDataFrameAnalyticsMemoryUsage', { body: request.payload })