This looks like a good basic configuration.

You should add a note that this is for a read-heavy ES cluster.

If you have a write-heavy cluster (thinking ELK), you'll want a different config with more focus on bulk and index.