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

Print a warning when Discovery SPI enabled but not configured on client #9741

Merged
merged 1 commit into from Jan 25, 2017
Merged

Print a warning when Discovery SPI enabled but not configured on client #9741

merged 1 commit into from Jan 25, 2017

Conversation

mdogan
Copy link
Contributor

@mdogan mdogan commented Jan 24, 2017

Printing a warning log instead of throwing exception to make similar
behavior with server side.

Fixes #9722

@mdogan
Print a warning when Discovery SPI enabled but not configured on client
e6d8b49 
Printing a warning log instead of throwing exception to make similar
behavior with server side.

Fixes hazelcast#9722
@mdogan mdogan added this to the 3.8 milestone Jan 24, 2017
@devOpsHazelcast
Copy link
Collaborator

Test PASSed.

@pveentjer
Copy link
Contributor

👍

@mdogan mdogan merged commit 683dab5 into hazelcast:master Jan 25, 2017
@mdogan mdogan deleted the issue-9722 branch January 25, 2017 07:55
@mdogan mdogan mentioned this pull request Jan 26, 2017
Merged
@mmedenjak mmedenjak added the Source: Internal PR or issue was opened by an employee label Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Source: Internal PR or issue was opened by an employee Team: Client Type: Defect
Projects
None yet
Milestone
3.8
Development

Successfully merging this pull request may close these issues.

Client side discovery assumes DiscoveryStrategyConfigs
4 participants
@mdogan @devOpsHazelcast @pveentjer @mmedenjak