nw9663644-eng commented on PR #370:
URL: https://github.com/apache/hugegraph-ai/pull/370#issuecomment-4966004296
> Blocking: yes. Summary: The latest head fixes the previously reported
concurrency issues, but malformed container shapes can still lose chunk-level
failure context; the existing malformed-response thread already covers this, so
no duplicate inline comment is added. Evidence: static review of
property_graph_extract.py lines 170-182 and 203-204; all visible latest-head
checks are green.阻塞：是。摘要：最新版本修复了之前报告的并发问题，但格式错误的容器结构仍然可能丢失块级故障上下文；现有的
malformed-response 讨论串已涵盖此问题，因此不再添加重复的内联注释。证据：对 property_graph_extract.py 文件第
170-182 行和 203-204 行进行静态审查；所有可见的最新版本检查结果均为绿色。
Thanks for the review. I fixed this by validating the property graph
container shape during per-chunk response parsing.
`vertices` and `edges` now must both be lists before the response can
proceed to filtering. Malformed container shapes such as `{"vertices": {},
"edges": []}` now fail with chunk context instead of being silently merged as
an empty result.
I also added regression coverage for malformed container shapes.
--
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]
