Skip to content

Conversation

@jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Jan 9, 2020

Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

  • 'Test an analyzer'
  • 'Configuring built-in analyzers'
  • 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.

Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

* 'Test an analyzer'
* 'Configuring built-in analyzers'
* 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.
@jrodewig jrodewig added >docs General docs changes :Search Relevance/Analysis How text is split into tokens labels Jan 9, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search (:Search/Analysis)

@debadair
Copy link
Contributor

This also sets us up for having a bunch of "Configuring XYZ" topics with little actual information. Do we really need a bucket for a feature's task-oriented topics?

@jrodewig
Copy link
Contributor Author

This also sets us up for having a bunch of "Configuring XYZ" topics with little actual information.

While it's currently only three paragraphs, there is useful information on this page.
Specifically, it:

  • Points out that users may not need to do any configuration at all
  • Suggests an order for trying out analyzers: Standard (default) -> Built-in analyzers -> Custom analyzers
  • Provides a list of available tutorials that can be scanned quickly.

Do we really need a bucket for a feature's task-oriented topics?

Well, I did open the PR. 😄

As stated in this comment on #50801, I believe there are advantages to explicitly separating documentation based on its intended use. If those aren't satisfactory, more specific questions may be helpful.

@jrodewig jrodewig merged commit 0fa6ac0 into elastic:master Jan 16, 2020
@jrodewig jrodewig deleted the add-configure-text-analysis branch January 16, 2020 18:11
jrodewig added a commit that referenced this pull request Jan 16, 2020
Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

* 'Test an analyzer'
* 'Configuring built-in analyzers'
* 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.
jrodewig added a commit that referenced this pull request Jan 16, 2020
Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

* 'Test an analyzer'
* 'Configuring built-in analyzers'
* 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.
jrodewig added a commit that referenced this pull request Jan 16, 2020
Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

* 'Test an analyzer'
* 'Configuring built-in analyzers'
* 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.
@jrodewig
Copy link
Contributor Author

Backport commits

master: 0fa6ac0
7.x: 7ef906f
7.6: 7348dfc
7.5: 5440af3

SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
Adds a 'Configure text analysis' page to house tutorial content for the
analysis topic.

Also relocates the following pages as children as this new page:

* 'Test an analyzer'
* 'Configuring built-in analyzers'
* 'Create a custom analyzer'

I plan to add a tutorial for specifying index-time and search-time
analyzers to this section as part of a future PR.
@polyfractal polyfractal added v7.6.0 and removed v7.6.1 labels Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants