-
Notifications
You must be signed in to change notification settings - Fork 81
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
Rollup prints DEPENDENCY_RELEASED
error to console when groupings are added
#6056
Comments
Seeing a few errors in the debug logs:
Full log:
|
Currently affecting our e2e tests: https://github.com/deephaven/web-client-ui/actions/runs/10832101472/job/30055965940?pr=2217 |
The issue has to do with changing the viewport - we're re-creating the viewport instead of just adjusting it, and part of the creation process is letting the old table reference be released on the server. This fix will improve tree performance, since (like tables) we will no longer create new subscriptions for each scroll change. |
Nate has reproduced this - we can't see it so far in using websocket transport, but using h2 and our emulated bidi streams (and no jvm debugger) we can cause it. With the debugger, we can't easily cause it, but we can see that the view is being released very shortly after creation. |
When a necessary operation is still running, clients must not release upstream tickets. This has mostly worked by the server running fast enough or by the websocket transport serializing operations in a way that we can't always rely on. Testing this is difficult to reliably do - the ticket has some manual steps that generally make it possible to see the issue, and we're exploring other options to make bugs like this more obvious. At this time, there is no good way to put an integration test in that verifies correct behavior efficiently. Fixes deephaven#6056 Fixes DH-18486
Description
Seems to be introduced with the JS API refactoring changes, though unsure if this is because we were misusing the API prior somehow.
Steps to reproduce
Expected results
2. Rollup table should appear, no error printed to console
Actual results
2. Rollup table appears correctly, but there is an error printed to the worker console:
Additional details and attachments
Added support logs with the JS API printing debug statements as well to help:
2024-09-11-163428_support_logs.zip
Versions
The text was updated successfully, but these errors were encountered: