Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOCS] Adds new transform limitation item and a note to the tutorial #79479

Merged
merged 8 commits into from
Oct 26, 2021

Conversation

szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Oct 19, 2021

Overview

Some of the transform configuration options are available only via the APIs, Kibana doesn't support them. For example, it is possible to set an ingest pipeline as dest by using the PUT transform API, but it's not possible on the UI.

This PR:

  • adds a note about this fact to step 5 of the tutorial to remind users,
  • adds an important note to the Overview page,
  • adds an item to the UI-related limitations about it.

Preview

Copy link

@hendrikmuhs hendrikmuhs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM regarding the added content to existing pages.

WDYT about some enhancement on the overview page? We do not communicate clearly that there is a UI and an API, that both can be used depending on what the users want to solve. Such an introduction could repeat the information that the API has more than the UI.

When talking with users, a lot of them don't know both exists and what are the differences.

@szabosteve
Copy link
Contributor Author

@hendrikmuhs
I think it's a good idea to add a short note to the Overview page as well, as probably this is the place where we can reach most of the users. I updated via 24d9528.

@szabosteve
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Oct 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@szabosteve szabosteve added the >docs General docs changes label Oct 21, 2021
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Oct 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

weizijun added a commit to weizijun/elasticsearch that referenced this pull request Oct 26, 2021
* upstream/master: (209 commits)
  Enforce license expiration (elastic#79671)
  TSDB: Automatically add timestamp mapper (elastic#79136)
  [DOCS]  `_id` is required for bulk API's `update` action (elastic#79774)
  EQL: Add optional fields and limit joining keys on non-null values only (elastic#79677)
  [DOCS] Document range enrich policy (elastic#79607)
  [DOCS] Fix typos in 8.0 security migration (elastic#79802)
  Allow listing older repositories (elastic#78244)
  [ML] track inference model feature usage per node (elastic#79752)
  Remove IncrementalClusterStateWriter & related code (elastic#79738)
  Reuse previous indices lookup when possible (elastic#79004)
  Reduce merging in PersistedClusterStateService (elastic#79793)
  SQL: Adjust JDBC docs to use milliseconds for timeouts (elastic#79628)
  Remove endpoint for freezing indices (elastic#78918)
  [ML] add timeout parameter for DELETE trained_models API (elastic#79739)
  [ML] wait for .ml-state-write alias to be readable (elastic#79731)
  [Docs] Update edgengram-tokenizer.asciidoc (elastic#79577)
  [Test][Transform] fix UpdateTransformActionRequestTests failure (elastic#79787)
  Limit CS Update Task Description Size (elastic#79443)
  Apply the reader wrapper on can_match source (elastic#78988)
  [DOCS] Adds new transform limitation item and a note to the tutorial (elastic#79479)
  ...

# Conflicts:
#	server/src/main/java/org/elasticsearch/index/IndexMode.java
#	server/src/test/java/org/elasticsearch/index/TimeSeriesModeTests.java
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :ml/Transform Transform Team:Docs Meta label for docs team Team:ML Meta label for the ML team v7.16.0 v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants