Skip to content

Change level of the log for unrecognized events to fine DEX-439 #1292

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.

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ihsandemir
Copy link
Collaborator

@ihsandemir ihsandemir commented May 27, 2025

Changed the level of the log for unrecognized events to fine in sync with the Java client.

See client protocol PR

Re-generated and updated codecs. This required introduction of the new class version for cluster version.

fixes #1293

…with the Java client.

See client protocol [PR](hazelcast/hazelcast-client-protocol#551)

Re-generated and updated codecs. This required introduction of the new class `version` for cluster version.
@ihsandemir ihsandemir added this to the 5.4.0 milestone May 27, 2025
@ihsandemir ihsandemir requested a review from JackPGreen May 27, 2025 09:16
@ihsandemir ihsandemir self-assigned this May 27, 2025
@ihsandemir ihsandemir changed the title Change level of the log for unrecognized events to fine Change level of the log for unrecognized events to fine [DEX-439] May 27, 2025
@ihsandemir ihsandemir changed the title Change level of the log for unrecognized events to fine [DEX-439] Change level of the log for unrecognized events to fine DEX-439 May 27, 2025
Comment on lines +61 to +76
/*
bool operator==(const version& rhs) const;

bool operator!=(const version& rhs) const;

bool operator<(const version& rhs) const;

bool operator>(const version& rhs) const;

bool operator<=(const version& rhs) const;

bool operator>=(const version& rhs) const;

friend std::ostream HAZELCAST_API& operator<<(std::ostream& os,
const version& version);
*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we adding newly-commented out code?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

forgot to mention PR is still draft. let mw change the status. i will complete this part.

@ihsandemir ihsandemir marked this pull request as draft May 27, 2025 09:47
@ihsandemir ihsandemir removed this from the 5.4.0 milestone Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The client prints Unknown message type WARNING message every 30 seconds for the newly added addclusterviewlistener events
2 participants