imbajin opened a new pull request, #3313: URL: https://github.com/apache/hugegraph/pull/3313
## Purpose of the PR After a Store pod is replaced, an existing Server can continue using stale Store channels for graph scans. Unary RPC failures retire the affected node and channels, while scan failures previously only propagated the iterator error. Related [Server acceptance report](https://github.com/hugegraph/hugegraph-toolchain/pull/47#issuecomment-6065397279). ## Main Changes - Reuse the existing expected-node invalidation path when a scan receives `UNAVAILABLE`, allowing later scans to rebuild their channel bindings. - Preserve the original scan error, healthy nodes after cancellation or invalid requests, and same-address replacement nodes after late failure notifications. - Add regression coverage for channel retirement and rebinding, error propagation, and replacement preservation. ## Verifying these changes - [ ] Trivial rework / code cleanup without any test coverage. (No Need) - [ ] Already covered by existing tests. - [x] Need tests and can be verified as follows: - Scanner and shared channel-pool regression tests, Java 17 compilation, formatting and style checks passed. - In an authenticated single-node deployment, only vertex scans were issued after replacing the Store pod. Subsequent scans recovered on the same Server process; full graph data, schema and index readback matched the baseline. The first scan still timed out. - Independent code and runtime-evidence reviews completed. The original Server image was restored after validation. PD-only restart, multi-node recovery and other streaming query paths remain outside this validation. ## Does this PR potentially affect the following parts? - [ ] Dependencies - [ ] Modify configurations - [ ] The public API - [x] Other affects (Store client stream connection recovery) - [ ] Nope ## Documentation Status - [ ] `Doc - TODO`: required documentation is pending; complete it before merging. - [ ] `Doc - Done`: documentation is included here or linked below. - [x] `Doc - No Need`: internal client connection invalidation. Documentation files in this PR or paired hugegraph-doc PR: Same repair branch as [org PR #290](https://github.com/hugegraph/hugegraph/pull/290); upstream submission does not imply merged or upstream-mainline validation. -- 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]
