Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Choose default port numbers to reduce chances of conflicts #474

Merged
merged 2 commits into from
Dec 18, 2020
Merged

Choose default port numbers to reduce chances of conflicts #474

merged 2 commits into from
Dec 18, 2020

Conversation

dutor
Copy link
Contributor

@dutor dutor commented Dec 11, 2020

Rule of thumb to choose ports:

  • Not registered currently in IANA, so we have chance to occupy
  • Not in the ephermeral port range, to reduce issues like conflicts and simultaneous open
  • Catchy

@dutor dutor requested a review from a team December 11, 2020 08:48
@dutor dutor added the ready-for-testing PR: ready for the CI test label Dec 11, 2020
@amber-moe amber-moe added the doc affected Solution: improvements or additions to documentation label Dec 14, 2020
@yixinglu yixinglu merged commit 267dbcd into vesoft-inc:master Dec 18, 2020
@Shylock-Hg
Copy link
Contributor

Really meaningful number 💯

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@laura-ding laura-ding laura-ding approved these changes

@amber-moe amber-moe amber-moe approved these changes

@jievince jievince jievince approved these changes

Assignees
No one assigned
Labels
doc affected Solution: improvements or additions to documentation ready-for-testing PR: ready for the CI test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@dutor @Shylock-Hg @jievince @amber-moe @laura-ding @yixinglu