Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Prevent direct enumeration of changing collection - #947

Merged
Mikhail Arkhipov (MikhailArkhipov) merged 35 commits into
microsoft:masterfrom
MikhailArkhipov:946
Apr 12, 2019
Merged

Mikhail Arkhipov (MikhailArkhipov) merged 35 commits into
microsoft:masterfrom
MikhailArkhipov:946

Conversation

@MikhailArkhipov

Copy link
Copy Markdown

Fixes #946

Direct enumeration of documents can fail since collection may be changing as user opens and closes documents or analysis brings new documents via imports.

MikhailArkhipov added 30 commits March 1, 2019 09:51
This reverts commit 7ffc9db.
Comment thread src/Analysis/Ast/Impl/Documents/RunningDocumentTable.cs Outdated
@MikhailArkhipov
Mikhail Arkhipov (MikhailArkhipov) merged commit 6e3bbcc into microsoft:master Apr 12, 2019
Jake Bailey (jakebailey) pushed a commit to jakebailey/python-language-server that referenced this pull request Nov 1, 2019
* Fix microsoft#668 (partial)

* Tests

* Revert "Tests"

This reverts commit 7ffc9db.

* Exp

* Limit concurrency

* Concurrency limit

* Drop cache after analysis

* Fix regression

* Fix test

* Prevent direct enumeration of changing collection

* Add lock

* Change to a method
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Analyzing in background" forever

3 participants