-
Notifications
You must be signed in to change notification settings - Fork 1.9k
IGNITE-4501: Improvement of connection in a cluster of new node #1676
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
Closed
Closed
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
0e77682
Add using CLUSTER_REGION_ID for ordering nodes in ring
SharplEr c438948
Add tests for RegionNodeComparator
SharplEr 5a7d5e3
Add test to suite
SharplEr cdcda32
A little clean up code
SharplEr 620be44
Change ordering inside nodes field, and add maxNode field to avoid us…
SharplEr b45bad8
Add filed to TcpDiscoveryNode for faster getting cluster_region_id
SharplEr d9b68dd
Fix bug in RegionNodeComparator
SharplEr f36f07d
Cleanup code in RegionNodeComparatorTest
SharplEr 145394c
Add catch NumberFormatException
SharplEr eeaf244
cleanup TcpDiscoveryNodesRing
SharplEr a3431de
Code cleanup
SharplEr a41ee66
Add @NotNull to RegionNodeComparator.compare
SharplEr 8ebd887
Add @Nullable to TcpDiscoveryNode.getClusterRegionId
SharplEr fd91a18
cleanup modules/core/src/main/java/org/apache/ignite/spi/discovery/tc…
SharplEr f4652c8
Remove using cast to Nubmer
SharplEr 6a3ac2e
restore code format
SharplEr ecb82ff
restore code format2
SharplEr 2e3ec77
fix reusing variable
SharplEr 7705554
Fix lost regionId after deserialization
SharplEr 30d3cd9
Add test for save sorting
SharplEr 3e204be
Add RegionTcpDiscoverySelfTest
SharplEr f1bdd6c
Add license
SharplEr 5e9343b
Add RegionTcpDiscoverySelfTest to suite
SharplEr 907facf
Remove maxNode
SharplEr 33b8700
Merge RegionTcpDiscoverySelfTest and TcpDiscoverySelfTest
SharplEr 951d838
Hmm, i think i has added it before.
SharplEr 9d2b6bb
Add logic for using old behavior while old nodes still in topology
SharplEr c3a6270
Remove Java 8 methods like List.sort()
SharplEr 1951e4f
Cleanup code in TcpDiscoveryNodesRing
SharplEr 5515d05
format code
SharplEr a3ff2c1
More code format
SharplEr ce7a8f3
remove firstAdd from TcpDiscoveryNodesRing
SharplEr e40ae0a
Fix converted map types
SharplEr d4ad9ef
First iteration of fixing code after code review
SharplEr 92b1c84
Add new test
SharplEr fc354b6
Merge branch 'master' into ignite-4501
SharplEr 7d22a68
Fix javadoc
SharplEr dd16ce6
Fix version control
SharplEr 2e7f830
Add License
SharplEr b22a32f
Backup TcpDiscoverySelfTest
SharplEr d2df15e
Fix conflict
SharplEr 18ca73e
Merge with master
SharplEr 2730699
Merge branch 'ignite-4501' of github.com:SharplEr/ignite into ignite-…
SharplEr 653295a
After CR
SharplEr 7dfaaaf
temp commit
SharplEr 87a63e0
Fix bug in ServerImpl
SharplEr 13466fb
decomposition #decomposition(...)
SharplEr 40adc71
Ops)
SharplEr 08deee3
par Fix bug of coplex ring
SharplEr 0ce5421
Refactoring in TcpDiscoveryNodeAddedMessage
SharplEr a19da15
Remove unused vars and args
SharplEr 083a4e9
Resorted nodes in topHist for fix problems in GridDhtPartitionsExchan…
SharplEr cd94b32
commit for Igor
SharplEr e727015
.
SharplEr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not use "0"? I like "0" better since I can put new nodes prior to nodes started with default config.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking it's more predictable behavior: all nodes with default configuration less or equal than nodes with region id.