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

graph concurrent share listing #10683

Merged
merged 2 commits into from
Dec 2, 2024
Merged

Conversation

butonic
Copy link
Member

@butonic butonic commented Nov 28, 2024

We now use 20 go routines to concurrently fetch resource information of shared resources in the graph.

@butonic butonic self-assigned this Nov 28, 2024
@butonic butonic force-pushed the graph-concurrent-share-listing branch 2 times, most recently from 8a8e5a2 to 546008d Compare November 28, 2024 14:34
@mmattel
Copy link
Contributor

mmattel commented Nov 28, 2024

Note, this adds a new envvar (the global one already exists). As the introduction version is v7, I need to do the changed envvar table update magic after merging 😄

@butonic butonic force-pushed the graph-concurrent-share-listing branch from 546008d to 8e92fa4 Compare November 28, 2024 15:32
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic force-pushed the graph-concurrent-share-listing branch from 8e92fa4 to 9fea5c3 Compare November 28, 2024 15:53
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
74.9% Duplication on New Code (required ≤ 50%)

See analysis details on SonarQube Cloud

@kobergj
Copy link
Collaborator

kobergj commented Dec 2, 2024

We do not care about the duplication right now.

@kobergj kobergj merged commit 7cf0fd5 into master Dec 2, 2024
3 of 4 checks passed
@butonic butonic deleted the graph-concurrent-share-listing branch December 2, 2024 15:51
ownclouders pushed a commit that referenced this pull request Dec 2, 2024
@micbar micbar mentioned this pull request Dec 17, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants