Skip to content

chore: resource tree call memory optimization#249

Merged
Ash-exp merged 1 commit into
developfrom
chore-resource-tree-opt
May 14, 2025
Merged

chore: resource tree call memory optimization#249
Ash-exp merged 1 commit into
developfrom
chore-resource-tree-opt

Conversation

@Ash-exp

@Ash-exp Ash-exp commented May 2, 2025

Copy link
Copy Markdown
Contributor

@Ash-exp Ash-exp self-assigned this May 2, 2025
@Ash-exp Ash-exp force-pushed the chore-resource-tree-opt branch from 606a1b6 to 5b861f9 Compare May 2, 2025 08:03
@Ash-exp Ash-exp changed the base branch from main to develop May 2, 2025 08:04
nishant-d
nishant-d suggested changes May 2, 2025
View reviewed changes
Comment thread kubelink/config/GlobalConfig.go Outdated
Comment thread kubelink/config/GlobalConfig.go Outdated
@Ash-exp

Ash-exp commented May 9, 2025

Copy link
Copy Markdown
Contributor Author

BenchmarkGetChildObjectsV2_ResourceByLimit_1_1: Sample size 1000

op 1 -
==========================================================================
=== Time taken to get child objects: 0.635388167
==========================================================================

op 2 -
==========================================================================
=== Time taken to get child objects: 0.382470167
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.3695985
==========================================================================

op 3 -
==========================================================================
=== Time taken to get child objects: 0.383045833
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.363911708
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.386471458
==========================================================================

BenchmarkGetChildObjectsV2_ResourceByLimit_1_1-8              3     377817361 ns/op    22693376 B/op      280518 allocs/op
PASS

BenchmarkGetChildObjectsV2_ResourceByLimit_3_4: Sample size 1000

op 1 -
==========================================================================
=== Time taken to get child objects: 0.554906167
==========================================================================

op 2 -
==========================================================================
=== Time taken to get child objects: 5.472853875
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.516210791
==========================================================================

BenchmarkGetChildObjectsV2_ResourceByLimit_3_4-8              2    2994538938 ns/op    22671076 B/op      279061 allocs/op
PASS

BenchmarkGetChildObjectsV2_ResourceByLimit_1_2: Sample size 1000

op 1 -
==========================================================================
=== Time taken to get child objects: 0.506552834
==========================================================================

op 2 -
==========================================================================
=== Time taken to get child objects: 0.676015333
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.536666959
==========================================================================

BenchmarkGetChildObjectsV2_ResourceByLimit_1_2-8              2     606362188 ns/op    22533020 B/op      279496 allocs/op
PASS

BenchmarkGetChildObjectsV2_ResourceByLimit_1_4: Sample size 1000

op 1 -
==========================================================================
=== Time taken to get child objects: 0.710262917
==========================================================================

op 2 -
==========================================================================
=== Time taken to get child objects: 0.673702916
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.666806334
==========================================================================

BenchmarkGetChildObjectsV2_ResourceByLimit_1_4-8              2     670265125 ns/op    22315556 B/op      280379 allocs/op
PASS

BenchmarkGetChildObjectsV2_ResourceByLimit_1_5: Sample size 1000

op 1 -
==========================================================================
=== Time taken to get child objects: 0.8075255
==========================================================================

op 2 -
==========================================================================
=== Time taken to get child objects: 0.973424917
==========================================================================
==========================================================================
=== Time taken to get child objects: 0.817693667
==========================================================================

BenchmarkGetChildObjectsV2_ResourceByLimit_1_5-8              2     895592208 ns/op    22183420 B/op      280793 allocs/op
PASS

@Ash-exp Ash-exp force-pushed the chore-resource-tree-opt branch from 2ed2823 to d54bc5c Compare May 14, 2025 17:15
@Ash-exp Ash-exp merged commit 52dd283 into develop May 14, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vikramdevtron vikramdevtron Awaiting requested review from vikramdevtron vikramdevtron is a code owner

@prakarsh-dt prakarsh-dt Awaiting requested review from prakarsh-dt prakarsh-dt is a code owner

@vivek-devtron vivek-devtron Awaiting requested review from vivek-devtron vivek-devtron is a code owner

1 more reviewer

@nishant-d nishant-d nishant-d approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Ash-exp Ash-exp

Labels

PR:Ready-to-Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Ash-exp @nishant-d