contrueCT commented on PR #3167: URL: https://github.com/apache/hugegraph/pull/3167#issuecomment-5392946433
### Codecov status clarification The reported 4.92 percentage-point project coverage drop is caused by incomplete coverage aggregation for the PR head, rather than by this PR removing coverage from thousands of existing lines. - Base commit `431f6e6`: 41.06% coverage from [5 upload sessions](https://api.codecov.io/api/v2/github/apache/repos/hugegraph/commits/431f6e6b431014feb5b50d27ea67180b4610f10b/). - Head commit `74e8701`: 36.15% coverage from only [1 upload session](https://api.codecov.io/api/v2/github/apache/repos/hugegraph/commits/74e8701920af2c528163ec896c146d761be7f162/). The [upload list](https://api.codecov.io/api/v2/github/apache/repos/hugegraph/commits/74e8701920af2c528163ec896c146d761be7f162/uploads/) contains only the HugeGraph-Server CI build. The Codecov steps for Commons, PD, Store, HStore, Server RocksDB, and Server HBase all found coverage files but then logged an empty token and HTTP 429 rate-limit errors. The uploader exits with status 0 by default, so the corresponding GitHub Actions jobs still appear green. Only the Server memory upload was accepted. Relevant logs: - [Commons](https://github.com/apache/hugegraph/actions/runs/32700666764/job/97351336906) - [PD, Store, and HStore](https://github.com/apache/hugegraph/actions/runs/32700666893) - [Server backends](https://github.com/apache/hugegraph/actions/runs/32700667141) Accordingly, `codecov/patch` passes at 60.60% against a 41.06% target, while `codecov/project` fails because it compares the single-session head report with a five-session base report. The 26 missing lines shown in the bot comment are therefore based on an incomplete coverage set and should not be treated as the complete patch-coverage result. This does not remove the need to review exception-path coverage independently. The new `ShortestPathTraverserTest` is included in `UnitTestSuite`, and the CI unit suite completed 629 tests with zero failures. The reviewer threads covering multi-label acquisition cleanup, primary/suppressed exceptions, wrapper cleanup, backward traversal, and super-node traversal are resolved. Related repository-level JaCoCo aggregation work is tracked in #3160 and #3161. The anonymous-upload 429 failures across the other workflows are additional CI infrastructure evidence, so I am not changing the functional implementation solely in response to this incomplete Codecov report. -- 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]
