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.

Sign up for GitHub

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

Jump to bottom

Fused Jet vertex ignores configured local parallelism for individual vertices [HZ-2493] #24683

Closed
k-jamroz opened this issue May 29, 2023 · 1 comment · Fixed by #24859
Closed

Fused Jet vertex ignores configured local parallelism for individual vertices [HZ-2493] #24683

k-jamroz opened this issue May 29, 2023 · 1 comment · Fixed by #24859
Assignees
@burakgok
Labels
Add to Release Notes Source: Community PR or issue was opened by a community user to-jira Type: Defect
Milestone
5.4.0

Comments

@k-jamroz
Copy link
Collaborator

Describe the bug

Fused Jet vertex ignores configured local parallelism and always uses default parallelism.

Expected behavior
Fused vertex should have the same parallelism as the original vertices being fused. Vertices are only fused when they have the same preferred local parellism (which can also be default).

To Reproduce

https://hazelcastcommunity.slack.com/archives/C015Q2TUBKL/p1685104062670239
@github-actions github-actions bot changed the title Fused Jet vertex ignores configured local parallelism for individual vertices Fused Jet vertex ignores configured local parallelism for individual vertices [HZ-2493] May 29, 2023
@github-actions
Copy link
Contributor

Internal Jira issue: HZ-2493

@k-jamroz k-jamroz added this to the 5.4.0 milestone May 29, 2023
@burakgok burakgok self-assigned this Jun 19, 2023
@burakgok burakgok mentioned this issue Jun 20, 2023
Merged
4 tasks
burakgok added a commit that referenced this issue Jul 3, 2023
@burakgok
Honor local parallelism of fused vertices [HZ-2493] (#24859)
1e26821 
Fixes #24683

In addition, the following members are renamed:
1. xform2vertex → transform2vertex: We should avoid using "x" for "trans" for the sake of readability.
2. findFusableChain() → findFusibleChain(): Fusible is the correct spelling of fusable.
burakgok added a commit to burakgok/hazelcast that referenced this issue Jul 3, 2023
@burakgok
Honor local parallelism of fused vertices [HZ-2493] [5.3.z]
a310077 
Backports hazelcast#24859
Fixes hazelcast#24683

In addition, the following members are renamed:
1. xform2vertex → transform2vertex: We should avoid using "x" for "trans" for the sake of readability.
2. findFusableChain() → findFusibleChain(): Fusible is the correct spelling of fusable.
@burakgok burakgok mentioned this issue Jul 3, 2023
Merged
4 tasks
burakgok added a commit that referenced this issue Jul 4, 2023
@burakgok
Honor local parallelism of fused vertices [HZ-2493] [5.3.z] (#24942)
916253a 
Backports #24859
Fixes #24683

In addition, the following members are renamed:
1. xform2vertex → transform2vertex: We should avoid using "x" for "trans" for the sake of readability.
2. findFusableChain() → findFusibleChain(): Fusible is the correct spelling of fusable.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@burakgok burakgok

Labels
Add to Release Notes Source: Community PR or issue was opened by a community user to-jira Type: Defect
Projects
None yet
Milestone
5.4.0
Development

Successfully merging a pull request may close this issue.

Honor local parallelism of fused vertices [HZ-2493] burakgok/hazelcast
2 participants
@burakgok @k-jamroz