redis7.0-chinese-annotated icon indicating copy to clipboard operation
redis7.0-chinese-annotated copied to clipboard

Published 20 hours ago • verified publisher icon CN-annotation-team

cluster.c 添加部分中文注释

Open huihuang-chen opened this issue 3 years ago • 6 comments

亲兄弟们，帮忙看看集群部分的注释有没有问题，我也是第一次阅读集群部分的源码，有问题我再改

huihuang-chen avatar Sep 23 '22 02:09 huihuang-chen

问一下，提交.c文件，它对应的头文件需不需要一起提交，还是分成两个PR

huihuang-chen avatar Sep 23 '22 02:09 huihuang-chen

问一下，提交.c文件，它对应的头文件需不需要一起提交，还是分成两个PR

属于同一个主题一起提交就好

我看到 diff 里有包含 rio 相关的内容，可以 rebase 一下最新代码吗

enjoy-binbin avatar Sep 23 '22 03:09 enjoy-binbin

你说的diff是说和redis/redis仓库的不同吗，我去看看

huihuang-chen avatar Sep 23 '22 03:09 huihuang-chen

不是 是和本仓库的主分支，例如你的 rio PR 已经先合并了，然后你这个 cluster PR 里包含了 rio 的内容 （已经被合并过的，或者新加的）

enjoy-binbin avatar Sep 23 '22 03:09 enjoy-binbin

好的，我看一下怎么弄

huihuang-chen avatar Sep 23 '22 03:09 huihuang-chen

看了一下，好像是最新的，后面提交的都改到当前仓库

huihuang-chen avatar Sep 23 '22 04:09 huihuang-chen

他意思是把与本次pr无关的用rebase合并起来，比如你之前的rio的提交出现在这里了，那就把他们和新的commit合并在一起。我一般是创建不同的分支发起不同的pr，然后被merge了之后就把那个分支删除就好了。

ncghost1 avatar Sep 25 '22 10:09 ncghost1

他意思是把与本次pr无关的用rebase合并起来，比如你之前的rio的提交出现在这里了，那就把他们和新的commit合并在一起。我一般是创建不同的分支发起不同的pr，然后被merge了之后就把那个分支删除就好了。

好的，谢谢，我大概明白你的意思了，第一次玩github多人配合开发，不懂这一块的操作，抱歉

huihuang-chen avatar Sep 26 '22 01:09 huihuang-chen

发现cluster.h文件的注释之前有个地方注释写的问题，对于clusterNode 的inbound_link和link的注释，之前是说inbound_link作为TCP服务端连接只做读出来，后面发现当两个节点在握手的时候，inbound_link会发送PING包，link会读取发送来的PING包，应该是在握手阶段，inbound_link也会做write

huihuang-chen avatar Sep 26 '22 03:09 huihuang-chen

好的

huihuang-chen avatar Sep 28 '22 07:09 huihuang-chen

好的，没有了，之后cluster.c再添加注释我再提交到这个pr上

huihuang-chen avatar Sep 30 '22 10:09 huihuang-chen