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.

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

Don't allow empty memberlist on cluster id changes[API-1590] #1452

Merged
merged 771 commits into from
Dec 30, 2022

Conversation

srknzl
Copy link
Member

@srknzl srknzl commented Dec 30, 2022

Superseeds #1448

srknzl and others added 30 commits July 30, 2021 10:32
* Add code sample checker

* Update code samples to add client.shutdown
* Add distinct aggregator and tests

* Update documentation
…ternal proxies (hazelcast#1023)

* Fix DistributedObjectsTest failure due to internal proxies

* Fix HazelcastClientTest
* Test adjustments around jet enabled config for backward compatbility tests

* Fix datatypetest.js

* Move createMapping to TestUtil and use it in all sql integration tests

* Fix code samples
* Make membership listener's methods optionsl in interface
* Add ignorePatterns for IDEs that give warning

* Merge eslint configs

* Add eslint config file for user code
* Add markdown linter

* Update DOCUMENTATION.md

Co-authored-by: Metin Dumandag <metindumandag@gmail.com>

* Add md001 and md040 and refactor

* Add lint-markdown:fix and fix header in documentation.md

Co-authored-by: Metin Dumandag <metindumandag@gmail.com>
@codecov
Copy link

codecov bot commented Dec 30, 2022

Codecov Report

Merging #1452 (2c1c218) into master (895b041) will increase coverage by 0.06%.
The diff coverage is 97.61%.

@@            Coverage Diff             @@
##           master    #1452      +/-   ##
==========================================
+ Coverage   93.39%   93.46%   +0.06%     
==========================================
  Files         466      466              
  Lines       16622    16628       +6     
  Branches     1350     1350              
==========================================
+ Hits        15524    15541      +17     
+ Misses        800      792       -8     
+ Partials      298      295       -3     
Impacted Files Coverage Δ
src/invocation/ClusterService.ts 97.36% <96.29%> (+2.49%) ⬆️
src/HazelcastClient.ts 93.44% <100.00%> (ø)
src/core/MembershipListener.ts 100.00% <100.00%> (ø)
src/listener/ClusterViewListenerService.ts 100.00% <100.00%> (ø)
src/network/Connection.ts 95.72% <100.00%> (+0.89%) ⬆️
src/network/ConnectionManager.ts 85.35% <100.00%> (+1.36%) ⬆️
src/proxy/MultiMapProxy.ts 98.46% <0.00%> (-0.52%) ⬇️
... and 1 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@srknzl srknzl merged commit 95754b5 into hazelcast:master Dec 30, 2022
srknzl added a commit to fatihozer0/hazelcast-nodejs-client that referenced this pull request Mar 24, 2023
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.

8 participants