Skip to content
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

New query during load of stacked bar chart causes "AbortError: BodyStreamBuffer was aborted" #2999

Closed
philrz opened this issue Feb 7, 2024 · 1 comment · Fixed by #2996
Closed
Labels
bug Something isn't working

Comments

@philrz
Copy link
Contributor

philrz commented Feb 7, 2024

Repro is with Zui commit 28f3f50.

This issue is most easily triggered with a large data set, so I've loaded these 24 files of GitHub data into a pool called github.

image

Then I click on Query Pool and while the stacked bar chart still shows "Loading..." I issue the following query against that pool:

repo.name=="duckdb/duckdb" type=="PullRequestEvent"
| count() by login:=actor.login
| sort -r count
| head 5

As shown in the attached video, once that query is issued an Unhandled Runtime Error AbortError: BodyStreamBuffer was aborted is surfaced.

Repro.mp4

Via binary search I found that this problem appeared at Zui commit 28f3f50 (associated with #2934). At commit 4399c1a that came right before that, there was still an Uncaught DOMException shown in DevTools, but it did not surface in the app as an error.

image

But then at commit 28f3f50 it looks like:

image

As @jameskerr said upon reviewing this evidence:

ok, so they got more specific about the error for some reason and that trigger our NextJS dev server to pop something up.

@philrz philrz added the bug Something isn't working label Feb 7, 2024
@philrz
Copy link
Contributor Author

philrz commented Feb 8, 2024

Verified in Zui commit 8088f5c.

As shown in the attached video, now when we repeat the repro steps the query is issued immediately despite the stacked bar chart still showing "Loading..." and errors are not surfaced in the app nor DevTools.

Verify.mp4

Thanks @jameskerr!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant