imbajin commented on PR #2993:
URL: https://github.com/apache/hugegraph/pull/2993#issuecomment-4243529235
后续建议 / 可单独跟进
=====================
这条不一定要阻塞当前 PR，但我建议后续单独跟进一下：
1. `HugeCountStrategy` 这次是基于较老版本的 TinkerPop `CountStrategy` 做本地 fork，然后补了
negative-bound 的保护。
2. 上游在后续还专门修过一次 `ConnectiveStep`（`and()/or()`）相关问题，也就是 `TINKERPOP-2911`。
3. 当前这个本地 fork 没有把那部分一起带进来，所以从长期维护角度看，HugeGraph 现在虽然修住了
`#2930/#2931/#2932`，但仍然继承了旧版 `CountStrategy` 在 `and()/or()` 组合过滤上的潜在遗留行为。
建议后续补一小个 follow-up：
- 把上游 `TINKERPOP-2911` 对 `ConnectiveStep` 的修复一起移植进来
- 同时补 2~4 个回归测试，至少覆盖下面这些形态：
- `__.and(__.out().count().is(0), __.in().count().is(0))`
- `__.or(__.out().count().is(0), __.in().count().is(0))`
- `__.where(__.or(__.out("none").out().count().is(0), __.has("none")))`
- `__.where(__.or(__.out("none").out().count().is(0),
__.has("none").count().is(0)))`
简单说：
- 当前 PR：修当前 issue family，没问题
- 后续 follow-up：把本地 fork 和上游已修过的 `ConnectiveStep` 行为再对齐一下，会更稳
--
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]
