Commit cf17553
authored
fix(interactive): fix the bug by passing empty STORE_COUNT setting in Groot charts (#4406)
<!--
Thanks for your contribution! please review
https://github.com/alibaba/GraphScope/blob/main/CONTRIBUTING.md before
opening an issue.
-->
## What do these changes do?
<!-- Please give a short brief about these changes. -->
The missing value for `STORE_COUNT` in configmap.yaml
(https://github.com/alibaba/GraphScope/blob/main/charts/graphscope-store/templates/configmap.yaml#L126)
caused the compiler to overlook multiple stores in the configuration.
Consequently, the compiler might incorrectly assume the system is
non-distributed, potentially omitting essential repartition operations
(if worker_num is also set to 1) in the physical plan. This PR addresses
the issue by properly setting the value of `STORE_COUNT`.
## Related issue number
<!-- Are there any issues opened that will be resolved by merging this
change? -->
Fixes #44021 parent 54a22fb commit cf17553
File tree
3 files changed
+6
-0
lines changed- charts/graphscope-store/templates
- frontend
- store
3 files changed
+6
-0
lines changedLines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
100 | 102 | | |
101 | 103 | | |
102 | 104 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
| 104 | + | |
| 105 | + | |
104 | 106 | | |
105 | 107 | | |
106 | 108 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
100 | 102 | | |
101 | 103 | | |
102 | 104 | | |
| |||
0 commit comments