double subscribing on certain hasura queries

[sqlAlchemyNoob2004]

Checked for duplicates

Yes - I've already checked

Is this a regression?

No - This is a new bug

Version

2.11.12

Describe the bug

It appears that we are issuing the same hasura subscriptions multiple times on certain endpoints.

Inspecting websocket traffic for subscription SubActivityDirectives I can see that on page load we are sending the same subscribe message twice:

Unless I'm missing something, these are two unique subscriptions to the same query with the same planId variable.

I can see future updates over the socket come in twice:

Looking at effects.ts it appears we import activityDirectivesDB which is one subscription but we also create another when we call getActivitiesForPlan (

aerie-ui/src/utilities/effects.ts

Lines 2896 to 2897 in 3f19529

	const query = convertToQuery(gql.SUB_ACTIVITY_DIRECTIVES);
	const data = await reqHasura<ActivityDirectiveDB[]>(query, { planId }, user);

).

Does this create a second? Should getActivitiesForPlan just use the activityDirectivesDB which has already been initialized?

I'm not super familiar with aerie so maybe this is by design, but it seems kinda wasteful. Could also be causing unneeded re-renders on data that hasn't actually changed.

Reproduction

just inspect traffic in devtools

Logs

No response

System Info

chrome

Severity

Minor

[duranb]

@sqlAlchemyNoob2004 Yeah, getActivitiesForPlan is more for getting the activities for the plan more immediately for processing plan export, whereas the subscription is to handle updates after loading the plan view. They definitely use the same query except one should just be a fetch vs a websocket. This is still good to look more in to in general though. Thank you!

[sqlAlchemyNoob2004]

@duranb wanted to update this and say this issue isn't limited to subscriptions

i'm seeing this for regular queries as well, most egregiously on the parsed dictionaries. the response of these queries can be pretty big, so it would be good to not double request them if not needed:

[duranb]

@sqlAlchemyNoob2004 nice find! My suspicion is it has something to do with https://github.com/NASA-AMMOS/aerie-ui/blob/develop/src/components/sequencing/SequenceEditor.svelte#L126. Might need to make more granular reactive statements to not capture unnecessary updates to svelte states.