Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
K
kb
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • granite
  • kb
  • Wiki
    • Code review
  • it桔子parse

it桔子parse · Changes

Page history
add code_review: it桔子parse authored Dec 09, 2021 by 蒋家升's avatar 蒋家升
Show whitespace changes
Inline Side-by-side
Showing with 67 additions and 0 deletions
+67 -0
  • code-review/it桔子parse.md code-review/it桔子parse.md +67 -0
  • No files found.
code-review/it桔子parse.md 0 → 100644
View page @ 85a94ea4
## 评审目标
```buildoutcfg
1. 实现方案的正确性
2. 代码的坏味道
3. 规范性
```
### 评审日期
```
2021-12-09
```
### 评审人
```
蒋家升、范召贤
```
### 被评审人
```
李子健
```
### 参考链接
- [代码入口](http://tech.pingansec.com/granite/project-collie-app/-/tree/master/app_juzi_data)
### 流程
```buildoutcfg
1. 被评审人需先整体描述需要解决的问题、解决流程 (被评审人讲解过程中,评审人可以记录问题,不要打断被评审者的思路)
2. 被评审人讲完,评审人和与会人员可以提问题
3. 评审人进行评审 (被评审者或者与会人员记录评审待改进的内容,有时并不是只针对被评审者,而是所有编码者)
4. 评审完成之后,落实待修改项,主要是缺陷和规范性
```
### 值得学习的地方
```buildoutcfg
暂无
```
### 建议 by蒋家升
1. udms太多,从 [爬虫业务](http://tech.pingansec.com/granite/kb/-/wikis/data_stream) 看大类其实就3个;<br>
应思考此处是否应该做成判断任务分发的形式:
1. 从sql开始就做groupby;
2. 大类udms中逻辑判断分发;
3. (沟通协调)以爬虫datatype等分组
2. sql/其它db 的轮子
3. 代码规范 [remark]
4. 逻辑优化 [remark]
---
### 改进落实
```buildoutcfg
时间:
改进人:
监督人:
```
Clone repository
  • README
  • basic_guidelines
  • basic_guidelines
    • basic_guidelines
    • dev_guide
    • project_build
    • 开发流程
  • best_practice
  • best_practice
    • AlterTable
    • RDS
    • azkaban
    • create_table
    • design
    • elasticsearch
    • elasticsearch
      • ES运维
    • logstash
View All Pages