-
Notifications
You must be signed in to change notification settings - Fork 461
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
Multi-Query results disappear from results pane randomly #1553
Comments
@abist do you have any thoughts here? I'd guess it has something to do with the optimizations we use to avoid rendering off-screen result grids. |
MSSQL Extension Version: 1.22.0 VSCODE version details: I got the same issue for multi-query results while scrolling. This appeared to be a rendering issue. Any thoughts/workaround welcome. Many thanks. |
Name: SQL Server (mssql) Version: 1.91.1 Same issue here. It is so bad that once I get to one of the result tables towards the bottom of the set, the results will disappear before all ten rows are visible. Are there any workarounds at all? |
We're working on a new query results pane that's react-based and should be handling virtualization better. Can you try this on the latest build with the modern features enabled? |
I have what I think is the latest, and I have the "preview features"
checked as far as I can tell, but I am not seeing any differences. I am
sure I am missing something. Can you point me to the right place?
Thanks,
[image: image.png]
Version: 1.49.1
Commit: 2f696aa95a13ccabd7e0a4431d1d759e3a7b8cfd
Date: 2024-08-13T17:56:45.095Z
VS Code: 1.82.0
Electron: 25.9.8
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: 11.4.183.29-electron.0
OS: Linux x64 6.8.0-47-generic
Garland Duvall
CTO
770-883-4643
***@***.***
www.activtech.com <http://activtech.com/>
Empowering Digital Supply Chains
…On Fri, Nov 1, 2024 at 3:03 PM Benjin Dubishar ***@***.***> wrote:
We're working on a new query results pane that's react-based and should be
handling virtualization better. Can you try this on the latest build with
the modern features enabled?
—
Reply to this email directly, view it on GitHub
<#1553 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BADDG6IWUXKGX2PFUAT7QX3Z6PF7ZAVCNFSM4J4MKPQ2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TENBVGI2DGMRTGM4A>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Let me try this again... I realize now that you were talking about the vsix. I downloaded it and went through ADS's File->Install Extension from VSIX Package and picked the package. As far as I can tell, it did nothing. I see nothing in the developer console either. So, I am going to need a bit of help here to upgrade ADS with the VSIX. |
Just to make sure we're on the same page, this extension is intended for use with VS Code, not Azure Data Studio. All of these features already exist in some form over in Azure Data Studio. To install this extension in VS Code, try clicking "Install" from this page or searching for "mssql" from within VS Code's extension view. |
Wow, Friday was not my best day (obviously!) I am just so used to using ADS, and its result window is so bad that I was thinking about it exclusively. Anyway, I have loaded this in VSCode, and the results are good so far. I've got a script that produces 42 result sets that I am using for testing. Fingers crossed, the scrolling seems to be working well. I have no idea if you know this or not, but the new pane does not seem to respect the following settings: Thank you for pointing me in the right direction. |
I didn't realize that; thanks for letting us know! I've created #18351 to track that specific issue. Also, I'm going to close this one since we've rewritten the entire experience since it's been filed. If this issue is occurring in the present query pane, please file a new bug so we can have clean and recent info to work with. Thanks! |
Steps to Reproduce:
This is a big issue when the purpose of the code is to locate the queries (generated by the cursor's dynamic sql) that are returning results.
thanks.
-IAN
The text was updated successfully, but these errors were encountered: