Skip to content
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

Forbid adding schema properties if they existed before #2496

Merged
merged 4 commits into from
Jan 17, 2023
Merged

Forbid adding schema properties if they existed before #2496

merged 4 commits into from
Jan 17, 2023

Conversation

abby-cyber
Copy link
Contributor

No description provided.

SuperYoko
SuperYoko previously approved these changes Jan 16, 2023

- 登录的用户必须拥有对应权限才能执行`ALTER TAG`语句。详情请参见[内置角色权限](../../7.data-security/1.authentication/3.role-list.md)。

- 确保要修改的属性不包含索引,否则`ALTER TAG`时会报冲突错误`[ERROR (-1005)]: Conflict!`。删除索引请参见 [drop index](../14.native-index-statements/6.drop-native-index.md)。

- 确保新增的属性名不与已存在或被删除的属性名同名,否则新增属性会失败。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

先DROP,再ADD或者CHANGE也不行吗?

@whitewum whitewum merged commit e7ab2e2 into vesoft-inc:master Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants