Skip to content

Solution to https://community.cesium.com/t/it-took-more-than-ten-seconds-to-exit/23354 problem #1664

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

newpeople123
Copy link

Dear Cesium Official Team,

Hello!

This pull request primarily addresses the issue discussed on the Cesium forum regarding this link It took more than ten seconds to exit.

The approach is as follows: it tracks all pending requests made by the UnrealAssetAccessor class, and when a UCesiumRasterOverlay or Cesium3DTileset instance is destroyed, it simultaneously cancels all pending requests associated with that instance.

Its limitation: for Cesium Ion map resources, it determines this in a hard-coded manner (e.g., for Bing Maps or Cesium World Terrain).

For a large open-source repository, this code is not particularly perfect. I do not have high expectations for the acceptance of this pull request, but I hope my approach is helpful to you, and I hope you will pay attention to this issue. This would make it more friendly for developers in regions with poor network conditions or those facing network instability.

Best regards.

@kring
Copy link
Member

kring commented May 2, 2025

Thanks @newpeople123! Do we have a signed Contributor License Agreement from you already?
https://github.com/CesiumGS/community/tree/main/CLAs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants