Skip to content

393 fix cached dataset load#395

Open
mudinthewater wants to merge 5 commits intorelease/5.3.2from
393-fix-cached-dataset-load
Open

393 fix cached dataset load#395
mudinthewater wants to merge 5 commits intorelease/5.3.2from
393-fix-cached-dataset-load

Conversation

@mudinthewater
Copy link
Copy Markdown
Collaborator

@mudinthewater mudinthewater commented Feb 5, 2026

Implements #393

Fixes an issue in getTopicsWithSessions interaction with persistence storage that caused a minimum of 15s delay before realtime sessions were stored.

old behavior: getTopicsWithSessions would check the dataset cache with getDatasets. The problem is that this is a race condition with MIO loads, as when it runs there are almost never any datasets loaded yet.

new behavior:

  1. Added an async function that will check the getDatasets cache every 1 second for up to 15 seconds before returning an empty set. If a new dataset is found during this iteraction, will check for another dataset 1s later in case MIOS are still loading.
  2. Added a new flag to getTopicsWithSessions which will toggle between the new behavior and the old behavior. This preserves the instant response on user interaction with the realtime session selection UI.

@jvigliotta jvigliotta force-pushed the 393-fix-cached-dataset-load branch from 56836d5 to 7667ef0 Compare April 23, 2026 23:06
…had thrown, using strict equality, some code cleanup and logic fixes to prevent loops

(cherry picked from commit d07ab8d)

merging in updates made in PR for main: #406
@jvigliotta jvigliotta added this to the 5.3.2 Patch Release milestone Apr 27, 2026
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