bitflicker64 commented on code in PR #857:
URL:
https://github.com/apache/hugegraph-toolchain/pull/857#discussion_r4375637783
##########
hugegraph-hubble/hubble-be/src/main/java/org/apache/hugegraph/service/graph/GraphService.java:
##########
@@ -97,6 +97,10 @@ public Vertex updateVertex(HugeClient client, String
vertexId, VertexEntity enti
this.checkParamsValid(client, entity, false);
GraphManager graph = client.graph();
Vertex vertex = this.buildVertex(client, entity);
+ if (vertex.id() != null) {
Review Comment:
Important: `AUTOMATIC` labels have Snowflake numeric IDs, but `buildVertex`
sets no ID for them, so their updates still go through
`updateVertexProperty(String, ...)`, which sends a quoted string ID that misses
the numeric vertex. Convert the ID as a number for `AUTOMATIC` too, as
`QueryService.getRealVertexId` does.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
