bug#38561: 27.0.50; Feature request: expose buffer size setting for process filters

[yyoncho]

Hi Eli,

Thank you for taking care of this!

I can confirm that changing the new variable value is yielding the same results.

I have a question on how am I supposed to use the new setting. Is it possible to set it per process?

I tried 

(with-current-buffer (process-buffer proc)

   (setq-local read-process-output-max (* 1024 1024)))

but it does not work. If it is not possible to use it per process then I think that I have 2 options:

1. Document it in lsp-mode's readme 

2. Override it with a setting in lsp-mode with a bigger value.

Both have clear advantages and disadvantages. Can you advise? 

Thanks,

Ivan

On Sat, Dec 21, 2019 at 10:49 AM Eli Zaretskii <eliz@gnu.org> wrote:

> From: yyoncho <yyoncho@gmail.com>
> Date: Tue, 10 Dec 2019 19:25:17 +0200
>
> Adding some performance numbers from lsp-mode project.
>
> (with-current-buffer "DemoApplication.java"
>   (benchmark-run 10 (lsp-request "workspace/symbol" (list :query "S"))))
>
> ;; Emacs from master
> => (4.203047857 0 0.0)
>
> ;; tested with emacs compiled with 1mb buffer.
> => (3.132925793 0 0.0)

Your wish has been granted, see the new variable
read-process-output-max.

I'd appreciate testing this with your use cases.