-
Notifications
You must be signed in to change notification settings - Fork 80
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. |
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: