imbajin commented on PR #826: URL: https://github.com/apache/hugegraph-toolchain/pull/826#issuecomment-6278204759
A scope check for the large diff, using the current identical diff in both PRs: | Path group | Added lines | Deleted lines | | --- | ---: | ---: | | Tests | 12,319 | 1,710 | | License/notice material | 14,412 | 1,107 | | Dependency lockfiles | 7,806 | 12,319 | | Remaining frontend code/configuration/examples | 8,500 | 8,441 | | Build, CI and browser acceptance | 3,004 | 256 | | Backend | 38 | 191 | | Other configuration/documentation | 262 | 87 | | **Total** | **46,341** | **24,111** | These are path-based groups, not a semantic attribution of every line. Tests, notices and lockfiles account for about 75% of additions. The remaining frontend group includes replacement of legacy graph/UI code, rather than 8,500 lines of net growth. Its largest new pieces are the shared graph controller/runtime, the Hubble adapter and React canvas; the G6 patch file also includes unified-diff context. The build/CI group includes 1,616 lines in the real browser acceptance runner and its helpers, so it should not all be described as cache infrastructure. Cache reuse itself is documented separately in the description. Temporary local acceptance scripts are outside the PR. The remaining work stays focused on confirmed regressions, agreed business acceptance and CI. Graphs with at most 200 total vertices and edges remain the primary case; this is not a new large-graph optimization effort. -- 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]
