-
Notifications
You must be signed in to change notification settings - Fork 23
best practice in using bigint data type #68
base: master
Are you sure you want to change the base?
Conversation
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
361fba7
to
8963a83
Compare
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to hide blockchain
in this article, because it is illegal in some countries.
zh/_index.md
Outdated
@@ -77,6 +77,7 @@ summary: 为开发者提供 TiDB 中文版开发指南。 | |||
- [分页的最佳实践](tidb-database-development-specification/best-practices-for-paging.md) | |||
- [唯一序列号生成方案](tidb-database-development-specification/unique-serial-number-generation-scheme.md) | |||
- [流程规范](tidb-database-development-specification/process-specification.md) | |||
- [TiDB 大整数最佳实践](tidb-database-development-specification/best-practice-in-using-bigint-data-type.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not need TiDB
in the index.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use database
instead tidb
or delete it ?
@@ -0,0 +1,106 @@ | |||
--- | |||
title: TiDB 大整数最佳实践 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not need TiDB
in the title.
@@ -0,0 +1,106 @@ | |||
--- | |||
title: TiDB 大整数最佳实践 | |||
summary: 介绍 大整数 在 TiDB 中的应用 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why added space around 大整数
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will delete the space
summary: 介绍 大整数 在 TiDB 中的应用 | ||
--- | ||
|
||
# TiDB 大整数最佳实践 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
现在大部分程序语言直接支持 64 bit 的内置类型,即直接支持相关运算。而 `大整数` 是超过 64 bit 的整数,一般是需要自己来实现。 | ||
|
||
有些场景,`大整数`是必需品(比如 **区块链** 应用,大整数是基础)。我们 TiDB 客户也有大整数的需求,这里根据几个成功案例,总结大整数的解决方案。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too colloquial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it
|
||
### 2. 字节序(Byte Order) | ||
|
||
字节序,大家应该都有了解。或者,网上资料、说明一大坨,自己看吧。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too colloquial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to add these to a new section, for example called Best Practices.
First-time contributors' checklist
What is changed, added or deleted? (Required)
Which TiDB version(s) do your changes apply to? (Required)
all tidb version which is suport binary type