-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[doc]add charset #1750
[doc]add charset #1750
Conversation
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.
Good
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.
Great Job
@@ -29,9 +29,13 @@ CREATE SPACE [IF NOT EXISTS] <space_name> | |||
|
|||
* _replica_factor_ | |||
|
|||
_replica_factor_ 表示副本数量。默认值是 1,集群建议为 3 | |||
_replica_factor_ 表示副本数量。默认值是 1,集群建议为 3。 |
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.
@panda-sheep 字符集会引入任何历史兼容性问题吗?
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.
If it is UTF8, there will be no compatibility problems.
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.
when charset compare and display #1782 merged, order by and string compare may be different
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
What changes were proposed in this pull request?
Add charset and collation doc.
code PR #1709