search

rossabaker / lsp-scala

Scala support for lsp-mode using metals
GNU General Public License v3.0
49 stars 6 forks source link

customize lsp-response-timeout #7

Closed hb9 closed 5 years ago

hb9 commented 6 years ago

Due to long startup-times of metals for larger projects the default value of lsp-response-timeout of 10 seconds is not enough. It took me some time to figure that out. Maybe you could mention the necessity of increasing that value in the readme?

If the lsp-response-timeout value is to small the time-out-error is interrupting the initialization which is causing that lsp--workspace-server-capabilities is not setup correctly.

rossabaker commented 5 years ago

Changes were made to metals to do more work after the initialization. I think this is no longer an issue. If I'm wrong, please reopen.

olafurpg commented 5 years ago

The initialize request should not take more than a few hundred milliseconds at most. The indexing work happens now after the initialized notification.