SebastianGruza commented on PR #3220: URL: https://github.com/apache/hugegraph/pull/3220#issuecomment-5742288507
Round 2 in 8e914e6f, for the note at `GraphManager.java:761`: the `usePD=false` path does not reach `ensureCluster` (`initMetaManager()` only runs from `loadMetaFromPD()`, i.e. under `usePD=true`), which I confirmed on the lab with a build of this head: with `usePD=false` the server starts with the default `pd.cluster` and with `pd.cluster=hg-test` (0 and 13 keys, as the prefixes imply) and no cluster-related line in the log. `connectMetaManager()` nevertheless got an explicit `usePD` gate and two tests covering a start with `usePD=false` after a graph bound `hg`; 9/9. Details and the log in the inline thread. -- 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]
