bitflicker64 commented on PR #3187: URL: https://github.com/apache/hugegraph/pull/3187#issuecomment-5549881527
The `hstore` failure on attempt 1 looks like a timing flake rather than anything in this PR, noting it here so it does not need diagnosing twice. One failure out of 814 tests: `VertexCoreTest.testAddVertexWithTtlAndTtlStartTime`, at [`VertexCoreTest.java:1300`](https://github.com/apache/hugegraph/blob/36811483a2040f70ca6923a288a8c28cad0086c0/hugegraph-server/hugegraph-test/src/main/java/org/apache/hugegraph/core/VertexCoreTest.java#L1300). The assertion there is `Assert.assertTrue(vertices.hasNext())` immediately after `Thread.sleep(1100L)`, checking a TTL vertex is still alive, so it depends on the sleep landing inside the remaining TTL window on a loaded runner. Attribution, for what it is worth: this branch changes three Compose files, `test-compose.sh`, the Docker entrypoint, its test and the docker README, and no Java. The master commit merged in here, 3681148 (#3182), added only `testQueryByPrimaryValuesAndPropsWithCachedVertex` to that file and left the TTL method alone. The same job flaked once on the previous head and passed on the retry. Attempt 2 is running now. Happy to open a separate issue for the TTL timing if it is worth tracking. -- 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]
