pruepei / influx-proxy

Other
6 stars 4 forks source link

add sharding support #5

Closed pingliu closed 7 years ago

pingliu commented 7 years ago

目前仅仅支持一个shards 如果有两个shards,直接返回

Sn0rt commented 7 years ago

引入新的包需要纳入govendor的管理,具体方法: 第一步,pull eleme的dev分支。 第二步,在你的dev分支上rebase eleme dev分支。 第三步,使用govendor工具将你引人的包放入vendor目录,参考:govendor fetch $path/to/pkg$。 第四步,以一个单独的commit提交你的引入的代码。

额外要注意的是授权信息

moooofly commented 7 years ago

@pingliu 建议你的开发分支的命名和功能相关,不要直接使用 dev 这个名字;

moooofly commented 7 years ago

目前仅仅支持一个shards 如果有两个shards,直接返回

@pingliu 作为第一次提及的概念,应该增加相应说明,否则别人很难理解你的 “shard” 是不是和别人理解的 shard 是一个意思;

按照我看过代码后的理解:这里的 shard 是指 hash ring 上的 node 概念,而按照代码的实现,一个 node 会对应 N 个 backend ;

另外,要求设计思路明确写出来,比如说“如果有两个shards,直接返回”,肯定有“因为”和“所以”,在设计思路明确的情况下,reviewer 能够准确的理解你的想法,进而确认代码实现是否符合预期,否则理解上可能会有偏差~

Sn0rt commented 7 years ago

下次,vendor用新的pr来推吧,这样可以减少影响review体验。