imbajin commented on PR #2993: URL: https://github.com/apache/hugegraph/pull/2993#issuecomment-4243528231
审查总结 =========== 这次改动针对 `#2930 / #2931 / #2932` 的修复方向是正确的，我这边没有发现会阻塞当前 PR 的新 correctness 问题。 我重点核对了以下几类影响面： - `count().is(P)` 在负边界下的语义是否恢复正确 - `where()` / `match()` / `repeat().until()` 这几条已报错路径是否一致 - `HugeCountStrategy` 替换默认 `CountStrategy` 后，是否会和 HugeGraph 现有的 `HugeCountStepStrategy` / provider 优化链产生明显冲突 - 负 upper bound 不再下推为非法 backend range 之后，是否会影响其他 traversal 行为 结论如下： - 当前 PR 已经修住了负边界导致的错误 `not(...)` 优化问题 - 当前 PR 已经避免了 `gte(-3)` 这类场景生成非法 range - 新增回归测试覆盖了这次 issue family 的核心路径，方向是够用的 - 目前没有看到会影响 API、schema、存储后端协议或配置层的风险 我会把一个“建议后续补强但不一定阻塞本 PR”的点，单独放到下一条评论里，方便作者和 reviewer 分开判断。 -- 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]
