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

Add check for load parameter #272

Merged
merged 3 commits into from
Nov 30, 2020
Merged

Add check for load parameter #272

merged 3 commits into from
Nov 30, 2020

Conversation

Linary
Copy link
Contributor

@Linary Linary commented Nov 27, 2020

No description provided.

@@ -134,6 +134,11 @@ load.file-mapping.vertex.id-fields-should-in-column-names=ID 列 {0} 必须都
load.file-mapping.edge.not-exist.label=不存在 label 为 {0} 的边映射
load.file-mapping.edge.source-fields-cannot-be-empty=起点 ID 列不能为空
load.file-mapping.edge.target-fields-cannot-be-empty=终点 ID 列不能为空
load.file-mapping.load-parameter.max-parse-error.illegal=导入参数 'max-parse-errors' 不合法
Copy link
Collaborator

Choose a reason for hiding this comment

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

also use chinese

@Linary Linary force-pushed the check-load-parameter branch from 0fffb66 to 77ab547 Compare November 30, 2020 03:29
javeme
javeme previously approved these changes Nov 30, 2020
@@ -134,6 +134,11 @@ load.file-mapping.vertex.id-fields-should-in-column-names=The all id fields {0}
load.file-mapping.edge.not-exist.label=The edge mapping doesn't exist with label {0}
load.file-mapping.edge.source-fields-cannot-be-empty=The source fields can't be empty
load.file-mapping.edge.target-fields-cannot-be-empty=The target fields can't be empty
load.file-mapping.load-parameter.max-parse-error.illegal=The load paramter 'max-parse-errors' is illegal

Choose a reason for hiding this comment

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

paramter -> parameter

@Linary Linary merged commit aa05111 into master Nov 30, 2020
@Linary Linary deleted the check-load-parameter branch November 30, 2020 04:11
Linary added a commit that referenced this pull request Nov 30, 2020
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.

4 participants