-
Notifications
You must be signed in to change notification settings - Fork 102
Postgres Server Crash #407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hey, thanks for sharing! Could you share the file you were working on? |
Ya sure I wasn't getting any auto complete which was why i was checking the logs |
Hi adding something extra for some reason after |
└─┐pgt_lsp::handlers::text_document::did_change{params=DidChangeTextDocumentParams { text_document: VersionedTextDocumentIdentifier { uri: Url { scheme: "file", cannot_be_a_base: false, username: "", password: None, host: None, port: None, path: "/home/bedgurl58/ciezkaPraca/api/essa.sql", query: None, fragment: None }, version: 86 }, content_changes: [TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 13 }, end: Position { line: 0, character: 13 } }), range_length: Some(0), text: "a" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 14 }, end: Position { line: 0, character: 14 } }), range_length: Some(0), text: "n" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 15 }, end: Position { line: 0, character: 15 } }), range_length: Some(0), text: "g" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 16 }, end: Position { line: 0, character: 16 } }), range_length: Some(0), text: "u" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 17 }, end: Position { line: 0, character: 17 } }), range_length: Some(0), text: "a" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 18 }, end: Position { line: 0, character: 18 } }), range_length: Some(0), text: "g" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 19 }, end: Position { line: 0, character: 19 } }), range_length: Some(0), text: "e" }, TextDocumentContentChangeEvent { range: Some(Range { start: Position { line: 0, character: 20 }, end: Position { line: 0, character: 20 } }), range_length: Some(0), text: "s" }] }} |
Thanks for sharing! Would it be possible to share a full log from opening the file to the crash? This way, we can repro it in tests easily. |
This video might be helpfull vod.mp4 |
That's indeed helpful. What client are you using? Is there anything special in your setup? The logs look like the ranges in the change that causes the crash are not related to the state before the change but to the previous within the batch of changes. The protocol does not allow that, and I am wondering where that comes from. In your video it looks like you were simply adding and removing characters in the same line. Does this happen if you type slower too? |
NVIM v0.12.0-dev-454+ga4f31857
postgres-language-server 0.7.0 And yes, for me this lsp is very bugged. It closes even when I type slowly. For example when I type Insert into cate. And I click enter to autocomplete as categories and the server shuts down. I have no problems with any of the other lsp and I use 13 of them in total. |
Uh oh!
There was an error while loading. Please reload this page.
Bug report
Describe the bug
The text was updated successfully, but these errors were encountered: