Skip to content

Feature/routing #2

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
Merged
merged 37 commits into from
May 2, 2018
Merged

Feature/routing #2

merged 37 commits into from
May 2, 2018

Conversation

auxten
Copy link
Contributor

@auxten auxten commented May 2, 2018

  • add analysisVendor.sh to generate dep tree png
  • add Envelope to RPC proto
  • add test case

auxten added 30 commits April 18, 2018 13:56
add vendor dir, simple build.sh, route package
4d02031
add space to align
c22c823
Merge branch 'develop' into feature/routing
e2ec1ff
remove .gitkeep
8fadd6b
add consistent hash
736656c
enlarge ring namespace to uint64 instead of uint32, use fnv64-a inste…
c4628fc 
…ad of crc32, fix test case
fix file name typo :-p
f5ab651
hash ring can store NodeInfo, all UT passed, Cheers
5c172b5
DHT by consistent hash RPC works now, need to implement more API
a7c2a40
format code, need some tools to do this automaticly
c11ea34
add .travis.yml for CI
e63071c
fix build
f955958
add reviewdog at Travis
d6012c2
add dep install
41d765e
fix missing goverage golint for Travis
3d9d6b3
fix build.sh use default GOOS GOARCH
0204c57
update Travis key
d76a6a0
for reviewdog test
ea85542
run test and goverage
b03602a
fix .travis.yml
757fa30
fix .travis.yml
ee047cf
fix travis.yml .....
b014d17
fix travis
3644774
try reviewdog
3216c2f
make golint happy
edf646f
fix golint issue, add necessary comment
f4daa08
fix UT issue, typo
ce2adce
use thunderdb api token
db9c714
add graphic dep analysis
5c238e6
add analysisVendor.sh to generate dep tree png
67fe296
@auxten auxten requested a review from laodouya May 2, 2018 10:56
leventeliu
leventeliu approved these changes May 2, 2018
View reviewed changes
Copy link
Contributor

@leventeliu leventeliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, approve.

@auxten auxten merged commit b853699 into develop May 2, 2018
@auxten auxten deleted the feature/routing branch May 2, 2018 11:10
@getlai getlai mentioned this pull request Nov 20, 2018
Closed
auxten pushed a commit that referenced this pull request Dec 18, 2018
@ConnorLi96
Merge pull request #2 from ConnorLi96/ConnorLi96-patch-1-1
e50f684 
Update README.md
@seiferxiii seiferxiii mentioned this pull request Jan 30, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leventeliu leventeliu leventeliu approved these changes

@xq262144 xq262144 Awaiting requested review from xq262144

@zeqing-guo zeqing-guo Awaiting requested review from zeqing-guo

@laodouya laodouya Awaiting requested review from laodouya

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@auxten @leventeliu