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

SQL Partition Aware review fixes #1188

Merged
merged 3 commits into from
Apr 5, 2023
Merged

SQL Partition Aware review fixes #1188

merged 3 commits into from
Apr 5, 2023

Conversation

akeles85
Copy link
Contributor

@akeles85 akeles85 commented Mar 31, 2023 •
edited by ihsandemir
Loading

I have fixed the reviews for SQL Partition Aware.

@akeles85 akeles85 added this to the 5.3.0 milestone Mar 31, 2023
@akeles85 akeles85 self-assigned this Mar 31, 2023
@akeles85 akeles85 mentioned this pull request Mar 31, 2023
Merged
@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

ihsandemir
ihsandemir reviewed Apr 3, 2023
View reviewed changes
Copy link
Collaborator

@ihsandemir ihsandemir left a comment

Choose a reason for hiding this comment

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

Just left a comment here, what do you think?

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@akeles85 akeles85 merged commit cc1719d into hazelcast:master Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ihsandemir ihsandemir ihsandemir approved these changes

@OzanCansel OzanCansel Awaiting requested review from OzanCansel

Assignees

@akeles85 akeles85

Labels
Type: Enhancement
Projects
None yet
Milestone
5.3.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@akeles85 @devOpsHazelcast @ihsandemir