Skip to content

Comments

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

Merged
javeme merged 1 commit intoapache:websitefrom
wangyao2016:fix-propertykey-body-website
May 13, 2022
Merged

fix: bad request body simple in propertykey.md#135
javeme merged 1 commit intoapache:websitefrom
wangyao2016:fix-propertykey-body-website

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.

3 participants