Skip to content

Conversation

@GlenRSmith
Copy link
Contributor

I doubt I've formatted the intra-doc link properly. If I could get a hand with that I'd appreciate it.

@GlenRSmith GlenRSmith added the >docs General docs changes label Feb 19, 2017

Many of the same strategies outlined for the reduction of total disk usage also
provide an improvement in the speed of indexing.
See: https://www.elastic.co/guide/en/elasticsearch/reference/current/tune-for-disk-usage.html
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd reformat this as:

Many of the strategies outlined in <<tune-for-disk-usage>> also
provide an improvement in the speed of indexing.

Thanks, @ clintongormley!
@GlenRSmith
Copy link
Contributor Author

@clintongormley Change made as suggested. Thank you!

@clintongormley clintongormley merged commit cf75c70 into 5.2 Feb 20, 2017
clintongormley pushed a commit that referenced this pull request Feb 20, 2017
* Add note and link to 'tune for disk usage'

* Changed formatting as suggested

Thanks, @ clintongormley!
@clintongormley
Copy link
Contributor

thanks @GlenRSmith - merged

clintongormley pushed a commit that referenced this pull request Feb 20, 2017
* Add note and link to 'tune for disk usage'

* Changed formatting as suggested

Thanks, @ clintongormley!
clintongormley pushed a commit that referenced this pull request Feb 20, 2017
* Add note and link to 'tune for disk usage'

* Changed formatting as suggested

Thanks, @ clintongormley!
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 20, 2017
* master:
  Mark IP range aggregator test as awaits fix
  Add note and link to 'tune for disk usage' (elastic#23252)
@colings86 colings86 deleted the doc-tune-index-moreinfo branch May 27, 2020 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants