TiDB 开源社区指南(上)

Easter79
• 阅读 764

作者:申砾

本系列文章旨在帮助社区开发者了解 TiDB 项目的全貌,更好的参与 TiDB 项目开发。大致会分两个角度进行描述:

  • 从社区参与者的角度描述如何更好的参与 TiDB 项目开发;

  • 从 PingCAP 内部团队的角度展示 TiDB 的开发流程,包括版本规划、开发流程、Roadmap 制定等。

希望通过一内一外两条线的描述,读者能在技术之外对 TiDB 有更全面的了解。本篇将聚焦在社区参与者的角度进行描述,也就是“外线”。

了解 TiDB

参与一个开源项目第一步总是了解它,特别是对 TiDB 这样一个大型的项目,了解的难度比较高,这里列出一些相关资料,帮助 newcomers 从架构设计到工程实现细节都能有所了解:

当然,最高效地熟悉 TiDB 的方式还是使用它,在某些场景下遇到了问题或者是想要新的 feature,去跟踪代码,找到相关的代码逻辑,在这个过程中很容易对相关模块有了解,不少 Contributor 就是这样完成了第一次贡献。 

我们还有一系列的 Infra Meetup,大约两周一次,如果方便到现场的同学可以听到这些高质量的 Talk。除了北京之外,其他的城市(上海、广州、成都、杭州)也开始组织 Meetup,方便更多的同学到现场来面基。

发现可以参与的事情

对 TiDB 有基本的了解之后,就可以选一个入手点。在 TiDB repo 中我们给一些简单的 issue 标记了 for-new-contributors 标签,这些 issue 都是我们评估过容易上手的事情,可以以此为切入点。另外我们也会定期举行一些活动,把框架搭好,教程写好,新 Contributor 按照固定的模式即可完成某一特性开发。

当然除了那些标记为 for-new-contributors 的 issue 之外,也可以考虑其他的 issue,标记为 help-wanted 标签的 issue 可以优先考虑。除此之外的 issue 可能会比较难解决,需要对 TiDB 有较深入的了解或者是对完成时间有较高的要求,不适合第一次参与的同学。

当然除了现有的 issue 之外,也欢迎将自己发现的问题或者是想要的特性提为新的 issue,然后自投自抢 :) 。 

当你已经对 TiDB 有了深入的了解,那么可以尝试从 Roadmap 上找到感兴趣的事项,和我们讨论一下如何参与。

讨论方案

找到一个感兴趣的点之后,可以在 issue 中进行讨论,如果是一个小的 bug-fix 或者是小的功能点,可以简单讨论之后开工。即使再简单的问题,也建议先进行讨论,以免出现解决方案有问题或者是对问题的理解出了偏差,做了无用功。

但是如果要做的事情比较大,可以先写一个详细的设计文档,提交到 docs/design 目录下面,这个目录下有设计模板以及一些已有的设计方案供你参考。一篇好的设计方案要写清楚以下几点:

  • 背景知识

  • 解决什么问题

  • 方案详细设计

  • 对方案的解释说明,证明正确性和可行性

  • 和现有系统的兼容性

  • 方案的具体实现

 

用一句话来总结就是写清楚“你做了什么,为什么要做这个,怎么做的,为什么要这样做”。如果对自己的方案不太确定,可以先写一个 Google Doc,share 给我们简单评估一下,再提交 PR。

提交 PR

按照方案完成代码编写后,就可以提交 PR。当然如果开发尚未完成,在某些情况下也可以先提交 PR,比如希望先让社区看一下大致的解决方案,这个时候请将 PR 标记为 WIP。

对于 PR 我们有一些要求:

  1. 需要能通过 make dev 的测试,跑过基本的单元测试;

  2. 必须有测试,除非只是改动文档或者是依赖包,其他情况需要有充足的理由说明没有测试的原因;

  3. 代码以及注释的质量需要足够高,这里 有一些关于编码风格和 commit message 的 guide;

  4. 请尽可能详细的填写 PR 的描述,并打上合适的 label。

对于 PR 的描述,我们提供了一个模板,希望大家能够认真填写,一个好的描述能够加速 PR 的 review 过程。通过这个模板能够向 reviewers 以及社区讲明白:

  • 这个PR 解决什么问题:相关的问题描述或者是 issue 链接;

  • 如何解决:具体的解决方法,reviewers 会根据这里的描述去看代码变动,所以请将这一段写的尽可能详细且有帮助;

  • 测试的情况;

  • 其他相关信息(如果需要):benchmark 结果、兼容性问题、是否需要更新文档。

最后再说几句测试,正确性是数据库安身立命之本,怎么强调测试都不为过。PR 中的测试不但需要充足,覆盖到所做的变动,还需要足够清晰,通过代码或者注释来表达测试的目的,帮助 reviewer 以及今后可能变动/破坏相关逻辑的人能够容易的理解这段测试。一段完善且清晰的测试也有利于让 reviewer 相信这个 Patch 是正确的。

PR review

PR review 的过程就是 reviewer 不断地提出 comment,PR 作者持续解决 comment 的过程。

每个 PR 在合并之前都需要至少得到两个 Committer/Maintainer 的 LGTM,一些重要的 PR 需要得到三个,比如对于 DDL 模块的修改,默认都需要得到三个 LGTM。

Tips:

  • 提了PR 之后,可以 at 一下相关的同学来 review;

  • Address comment 之后可以 at 一下之前提过 comment 的同学,标准做法是 comment 一下 “PTAL @xxx”,这样我们内部的 Slack 中可以得到通知,相关的同学会受到提醒,让整个流程更紧凑高效。

与项目维护者之间的交流

目前标准的交流渠道是 GitHub issue,请大家优先使用这个渠道,我们有专门的同学来维护这个渠道,其他渠道不能保证得到研发同学的及时回复。这也是开源项目的标准做法。

无论是遇到 bug、讨论具体某一功能如何做、提一些建议、产品使用中的疑惑,都可以来提 issue。在开发过程中遇到了问题,也可以在相关的 issue 中进行讨论,包括方案的设计、具体实现过程中遇到的问题等等。

最后请大家注意一点,除了 pingcap/docs-cn 这个 repo 之外,请大家使用英文。

更进一步

当你完成上面这些步骤的之后,恭喜你已经跨过第一个门槛,正式进入了 TiDB 开源社区,开始参与 TiDB 项目开发,成为 TiDB Contributor。

如果想更进一步,深入了解 TiDB 的内部机制,掌握一个分布式数据库的核心模块,并能做出改进,那么可以了解更多的模块,提更多的 PR,进一步向 Committer 发展(这里 解释了什么是 Committer)。目前 TiDB 社区的 Committer 还非常少,我们希望今后能出现更多的 Committer 甚至是 Maintainer。

从 Contributor 到 Committer 的门槛比较高,比如今年的新晋 Committer 杜川同学,在成为 Committer 的道路上给 tidb/tikv 项目提交了大约 80 个 PR,并且对一些模块有非常深入的了解。当然,成为 Committer 之后,会有一定的权利,比如对一些 PR 点 LGTM 的权利,参加 PingCAP 内部的技术事项、开发规划讨论的权利,参加定期举办的 TechDay/DevCon 的权利。目前社区中还有几位贡献者正走在从 Contributor 到 Committer 的道路上。

TiDB 开源社区指南(上)

点赞
收藏
评论区
推荐文章
blmius blmius
2年前
MySQL:[Err] 1292 - Incorrect datetime value: ‘0000-00-00 00:00:00‘ for column ‘CREATE_TIME‘ at row 1
文章目录问题用navicat导入数据时,报错:原因这是因为当前的MySQL不支持datetime为0的情况。解决修改sql\mode:sql\mode:SQLMode定义了MySQL应支持的SQL语法、数据校验等,这样可以更容易地在不同的环境中使用MySQL。全局s
Jacquelyn38 Jacquelyn38
2年前
2020年前端实用代码段,为你的工作保驾护航
有空的时候,自己总结了几个代码段,在开发中也经常使用,谢谢。1、使用解构获取json数据let jsonData  id: 1,status: "OK",data: 'a', 'b';let  id, status, data: number   jsonData;console.log(id, status, number )
皕杰报表之UUID
​在我们用皕杰报表工具设计填报报表时,如何在新增行里自动增加id呢?能新增整数排序id吗?目前可以在新增行里自动增加id,但只能用uuid函数增加UUID编码,不能新增整数排序id。uuid函数说明:获取一个UUID,可以在填报表中用来创建数据ID语法:uuid()或uuid(sep)参数说明:sep布尔值,生成的uuid中是否包含分隔符'',缺省为
Stella981 Stella981
2年前
JS 苹果手机日期显示NaN问题
问题描述newDate("2019122910:30:00")在IOS下显示为NaN原因分析带的日期IOS下存在兼容问题解决方法字符串替换letdateStr"2019122910:30:00";datedateStr.repl
Easter79 Easter79
2年前
Twitter的分布式自增ID算法snowflake (Java版)
概述分布式系统中,有一些需要使用全局唯一ID的场景,这种时候为了防止ID冲突可以使用36位的UUID,但是UUID有一些缺点,首先他相对比较长,另外UUID一般是无序的。有些时候我们希望能使用一种简单一些的ID,并且希望ID能够按照时间有序生成。而twitter的snowflake解决了这种需求,最初Twitter把存储系统从MySQL迁移
Easter79 Easter79
2年前
TiDB 的现在和未来
本文根据黄东旭在PingCAPD轮融资线上发布会的演讲实录进行整理。!(https://oscimg.oschina.net/oscnet/upceee1b1ad10e2bcde45dbe1cc00dc0688fc.png)TiDB的现在和未来大家好,我是黄东旭,是PingCAP的联合创始人和CTO
Wesley13 Wesley13
2年前
00:Java简单了解
浅谈Java之概述Java是SUN(StanfordUniversityNetwork),斯坦福大学网络公司)1995年推出的一门高级编程语言。Java是一种面向Internet的编程语言。随着Java技术在web方面的不断成熟,已经成为Web应用程序的首选开发语言。Java是简单易学,完全面向对象,安全可靠,与平台无关的编程语言。
Stella981 Stella981
2年前
Django中Admin中的一些参数配置
设置在列表中显示的字段,id为django模型默认的主键list_display('id','name','sex','profession','email','qq','phone','status','create_time')设置在列表可编辑字段list_editable
Wesley13 Wesley13
2年前
MySQL部分从库上面因为大量的临时表tmp_table造成慢查询
背景描述Time:20190124T00:08:14.70572408:00User@Host:@Id:Schema:sentrymetaLast_errno:0Killed:0Query_time:0.315758Lock_
Python进阶者 Python进阶者
3个月前
Excel中这日期老是出来00:00:00,怎么用Pandas把这个去除
大家好,我是皮皮。一、前言前几天在Python白银交流群【上海新年人】问了一个Pandas数据筛选的问题。问题如下:这日期老是出来00:00:00,怎么把这个去除。二、实现过程后来【论草莓如何成为冻干莓】给了一个思路和代码如下:pd.toexcel之前把这
Easter79
Easter79
Lv1
今生可爱与温柔,每一样都不能少。
文章
2.8k
粉丝
5
获赞
1.2k