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

Honor local parallelism of fused vertices [HZ-2493] #24859

Merged
merged 1 commit into from
Jul 3, 2023
Merged

Honor local parallelism of fused vertices [HZ-2493] #24859

merged 1 commit into from
Jul 3, 2023

Conversation

burakgok
Copy link
Contributor

@burakgok burakgok commented Jun 20, 2023 •
edited

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.

Checklist:

  • Labels (Team:, Type:, Source:, Module:) and Milestone set
  • Label Add to Release Notes or Not Release Notes content set
  • Request reviewers if possible
  • Send backports/forwardports if fix needs to be applied to past/future releases

@burakgok burakgok added Type: Defect Source: Internal PR or issue was opened by an employee Team: SQL Add to Release Notes Module: Jet Issues/PRs for Jet labels Jun 20, 2023
@burakgok burakgok added this to the 5.4.0 milestone Jun 20, 2023
@burakgok burakgok self-assigned this Jun 20, 2023
@burakgok burakgok removed the request for review from ivanthescientist July 3, 2023 12:04
@burakgok burakgok merged commit 1e26821 into hazelcast:master Jul 3, 2023
8 checks passed
burakgok added a commit to burakgok/hazelcast that referenced this pull request 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 pull request Jul 3, 2023
Merged
4 tasks
burakgok added a commit that referenced this pull request 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
Reviewers

@TomaszGaweda TomaszGaweda TomaszGaweda approved these changes

@Fly-Style Fly-Style Fly-Style approved these changes

Assignees

@burakgok burakgok

Labels
Add to Release Notes Module: Jet Issues/PRs for Jet Source: Internal PR or issue was opened by an employee Team: SQL Type: Defect
Projects
None yet
Milestone
5.4.0
Development

Successfully merging this pull request may close these issues.

Fused Jet vertex ignores configured local parallelism for individual vertices [HZ-2493]
3 participants
@burakgok @TomaszGaweda @Fly-Style