update to the max file descriptors number

[robb3rt]

• Have you signed the contributor license agreement?
• Have you followed the contributor guidelines?
• If submitting code, have you built your formula locally prior to submission with gradle check?
• If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
• If submitting code, have you checked that your submission is for an OS that we support?
• If you are submitting this code for a class then read our policy for that.

[elasticmachine]

Pinging @elastic/es-core-infra

[rjernst]

Thanks for the PR @robb3rt. Can you please explain why you believe this documentation needs updating? The docs match the bootstrap check we have in place for max file descriptors.

[robb3rt]

@rjernst the max file descriptors should be 65536 or higher or bootstrap checks will fail. Right now the commands we list to set the max file descriptors set it to 65535 instead.

Also quoting our documentation:

Make sure to increase the limit on the number of open files descriptors for
the user running Elasticsearch to 65,536 or higher.

[rjernst]

Which version of Elasticsearch are you using? This was changed to 65535 in #37537, which merged to 6.7 and 7.0. The documentation matches the check.

[rjernst]

I think the discrepancy is in the line that you quoted in your last comment, which should be changed to 65535. It was not caught in #37537 because it contains a comma.

[javanna]

hi @robb3rt do you plan on updating your PR based on the comments you received above?

[danielmitterdorfer]

I'm closing this PR meanwhile due to lack of feedback. If there is interest to continue with this PR, please reopen. Thank you. :)