Skip to content

Commit

Permalink
Revert "[ML] Return both modelId and inferenceId (elastic#111490)"
Browse files Browse the repository at this point in the history
Both `model_id` and `inference_id` were returned from the inference API
to support backwards compatibility while kibana migrates to
`inference_id`.

Now that the migration is complete, we will stop returning `model_id`.

This reverts commit 3b8970c.
  • Loading branch information
prwhelan committed Sep 4, 2024
1 parent 2982fc6 commit a3c73c4
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
if (params.paramAsBoolean(USE_ID_FOR_INDEX, false)) {
builder.field(INDEX_ONLY_ID_FIELD_NAME, inferenceEntityId);
} else {
builder.field(INDEX_ONLY_ID_FIELD_NAME, inferenceEntityId);
builder.field(INFERENCE_ID_FIELD_NAME, inferenceEntityId);
}
builder.field(TaskType.NAME, taskType.toString());
Expand All @@ -143,7 +142,6 @@ public XContentBuilder toFilteredXContent(XContentBuilder builder, Params params
if (params.paramAsBoolean(USE_ID_FOR_INDEX, false)) {
builder.field(INDEX_ONLY_ID_FIELD_NAME, inferenceEntityId);
} else {
builder.field(INDEX_ONLY_ID_FIELD_NAME, inferenceEntityId);
builder.field(INFERENCE_ID_FIELD_NAME, inferenceEntityId);
}
builder.field(TaskType.NAME, taskType.toString());
Expand Down

0 comments on commit a3c73c4

Please sign in to comment.