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

fix: bad request body simple in propertykey.md #135

Merged
merged 1 commit into from
May 13, 2022

Conversation

wangyao2016
Copy link
Contributor

@wangyao2016 wangyao2016 commented May 12, 2022

An error will be reported according to the original document, like:

Unrecognized field "property_key" (class com.baidu.hugegraph.api.schema.PropertyKeyAPI$JsonPropertyKey), not marked as ignorable (9 known properties: "check_exist", "properties", "aggregate_type", "id", "cardinality", "data_type", "name", "user_data", "write_type"]) at [Source: (org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream); line: 2, column: 23] (through reference chain: com.baidu.hugegraph.api.schema.PropertyKeyAPI$JsonPropertyKey["property_key"])

image

master branch pr

"task_id": 0
"name": "age",
"data_type": "INT",
"cardinality": "SINGLE"
Copy link
Contributor

Choose a reason for hiding this comment

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

This may be a mistake to treat the request body as the response body, since the "task_id" response format was changed in version 0.12, could you also please update Response Body.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for remind, done.

@wangyao2016 wangyao2016 force-pushed the fix-propertykey-body-website branch from 7dd5c4d to a9d95cd Compare May 13, 2022 06:10
@javeme javeme merged commit 1c933ca into apache:website May 13, 2022
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.

None yet

3 participants