Skip to content

Conversation

@mcollovati
Copy link
Contributor

Skip shutdown hook warning check when using a Hazelcast client, as ClientDynamicClusterConfig does not support reading properties and shutdown hook is not relevant for HazelcastClient.

Fixes #268

Description

Please list all relevant dependencies in this section and provide summary of the change, motivation and context.

Fixes # (issue)

Type of change

  • Bugfix
  • Feature

Checklist

  • I have read the contribution guide: https://vaadin.com/docs/latest/guide/contributing/overview/
  • I have added a description following the guideline.
  • The issue is created in the corresponding repository and I have referenced it.
  • I have added tests to ensure my change is effective and works as intended.
  • New and existing tests are passing locally with my change.
  • I have performed self-review and corrected misspellings.

Additional for Feature type of change

  • Enhancement / new feature was discussed in a corresponding GitHub issue and Acceptance Criteria were created.

Skip shutdown hook warning check when using a Hazelcast client,
as ClientDynamicClusterConfig does not support reading properties
and shutdown hook is not relevant for HazelcastClient.

Fixes #268
@mcollovati mcollovati requested a review from heruan January 12, 2026 12:02
@mcollovati mcollovati enabled auto-merge (squash) January 12, 2026 19:46
@mcollovati mcollovati merged commit befaa69 into main Jan 12, 2026
2 checks passed
@mcollovati mcollovati deleted the issues/268-prevent_failures_reading_hazelcast_config branch January 12, 2026 20:00
vaadin-bot pushed a commit that referenced this pull request Jan 12, 2026
Skip shutdown hook warning check when using a Hazelcast client,
as ClientDynamicClusterConfig does not support reading properties
and shutdown hook is not relevant for HazelcastClient.

Fixes #268
mcollovati added a commit that referenced this pull request Jan 12, 2026
Skip shutdown hook warning check when using a Hazelcast client,
as ClientDynamicClusterConfig does not support reading properties
and shutdown hook is not relevant for HazelcastClient.

Fixes #268

Co-authored-by: Marco Collovati <[email protected]>
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.

Problems with Hazelcast

3 participants