Skip to content

fix(client): refactor duplicated replica+partition logic #2925

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
Merged
dnwe merged 2 commits into from
Dec 19, 2024
Merged

fix(client): refactor duplicated replica+partition logic #2925

dnwe merged 2 commits into from
Dec 19, 2024

Conversation

@Trinoooo
Copy link
Contributor

@Trinoooo Trinoooo commented Jun 11, 2024

  1. extract the same logic in Replicas, InSyncReplicas, OfflineReplicas. Improve reusability, maintainability, and consistency
  2. remove unnecessary local variable and misleading annotion. see Sincerely ask, is this redundant? #2923

@dnwe dnwe force-pushed the style/eliminate_redundancies_and_misunderstandings branch from a1fb1e9 to 7dc5281 Compare August 7, 2024 16:51
puellanivis
puellanivis reviewed Aug 7, 2024
View reviewed changes
Copy link
Collaborator

@puellanivis puellanivis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks all reasonable to me.

@dnwe dnwe force-pushed the style/eliminate_redundancies_and_misunderstandings branch 4 times, most recently from 18da12b to fdfd7a4 Compare August 12, 2024 18:27
@dnwe dnwe changed the title style: eliminate redundancies and remove misleading annotion chore(client): refactor duplicate replica logic into shared func Aug 13, 2024
@dnwe dnwe changed the title chore(client): refactor duplicate replica logic into shared func fix(client): refactor duplicated replica+partition logic Aug 13, 2024
@dnwe dnwe force-pushed the style/eliminate_redundancies_and_misunderstandings branch from fdfd7a4 to 496e760 Compare August 13, 2024 21:54
@Trinoooo @dnwe
fix(client): refactor duplicated replica logic
8155116 
1. extract the same logic in Replicas, InSyncReplicas, OfflineReplicas.
   Improve reusability, maintainability, and consistency
2. remove unnecessary local variable and misleading annotation. see
   IBM#2923

Signed-off-by: Trino <[email protected]>
@Trinoooo @dnwe
fix(client): refactor duplicated partition logic
dada7ce 
extract the same logic in Partitions and WritablePartitions

Signed-off-by: Trino <[email protected]>
@dnwe dnwe force-pushed the style/eliminate_redundancies_and_misunderstandings branch from 496e760 to dada7ce Compare November 8, 2024 11:02
@dnwe dnwe merged commit 0757b96 into IBM:main Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@puellanivis puellanivis puellanivis left review comments

@dnwe dnwe dnwe approved these changes

Assignees

No one assigned

Labels

housekeeping

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Trinoooo @dnwe @puellanivis