Skip to content

Conversation

@dannypsnl
Copy link
Collaborator

I will use this branch for a few days before I make PR ready.

@dannypsnl
Copy link
Collaborator Author

Emmm, obviously not as easy as I thought.

@6cdh
Copy link
Collaborator

6cdh commented Nov 23, 2025

Yes. The parallel threads generally give good boost. But it slows down by 2x in my test for expand (expand 10 racket files). It probably won't benefit from parallel because of its own complexity.

@shhyou
Copy link
Contributor

shhyou commented Nov 24, 2025

If racket-langserver requires Racket >=9.0, I think it'd be better to name it racket-langserver2 or so.

@dannypsnl
Copy link
Collaborator Author

@shhyou Indeed, that's a problem.

I will close this PR, because the model, expansion is only going for the current editing file, where parallel thread can't help.

Also, I shall not force an upgrade, I think we can

  1. wait until Racket users all use >=9.0
  2. or create an experimental branch, creates a complete different interaction to documents

@dannypsnl dannypsnl closed this Nov 24, 2025
@dannypsnl dannypsnl deleted the use-parallel-thread branch November 24, 2025 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants