atharrva01 commented on code in PR #383:
URL: https://github.com/apache/hugegraph-ai/pull/383#discussion_r4291237205
##########
hugegraph-llm/src/hugegraph_llm/nodes/hugegraph_node/graph_query_node.py:
##########
@@ -364,7 +365,7 @@ def _subgraph_query(self, context: Dict[str, Any]) ->
Dict[str, Any]:
# TODO: use generator or asyncio to speed up the query logic
for matched_vid in matched_vids:
gremlin_query = VID_QUERY_NEIGHBOR_TPL.format(
- keywords=f"'{matched_vid}'",
+ keywords=gremlin_literal(matched_vid),
Review Comment:
Thanks for testing this against a real server, good catch. Fixed:
_process_vertex now uses str(matched_str) for the cached case. I also let the
fake endpoint return paths, and the numeric test now has two paths starting at
123, so it hits that branch. It fails with the same TypeError without the fix.
--
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]
