You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by "Cole-Greer (via GitHub)" <gi...@apache.org> on 2026/05/20 22:13:40 UTC

[PR] Fix Flaky .Net and server tests [tinkerpop]

Cole-Greer opened a new pull request, #3431:
URL: https://github.com/apache/tinkerpop/pull/3431

   `shouldTrackTransactionCountAccurately` was racing because commitTx()
   and rollbackTx() returned without consuming the HTTP response. With
   chunked transfer encoding, the client can receive response headers
   before the server completes transactionManager.destroy(). Wrapping
   in try-with-resources ensures the full response is read, guaranteeing
   server-side processing is complete before asserting the count.
   
   `shouldTimeoutIdleTransactionWithNoOperations` used a 1ms timeout,
   which could expire during the beginTx() HTTP round-trip itself,
   causing getFirstHeader(TRANSACTION_ID) to return null and NPE.
   Increased to 500ms — still well below the 1000ms sleep that
   validates timeout behavior.
   
   `PropertyDeserializationTests` and `GraphTraversalTests` were running
   concurrently with `@GraphComputerOnly` Gherkin scenarios. During OLAP
   computation, TinkerGraphComputer sets a graphComputerView on the
   shared TinkerGraph instance. Any concurrent read through TinkerVertex
   sees compute properties (like haltedTraversers) merged with real
   properties, causing spurious assertion failures and NPEs.
   
   Place all tests that hit the shared server into a single non-parallel
   xUnit collection (GremlinServerTests) so they never overlap with an
   active GraphComputer view.
   
   VOTE +1


-- 
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: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix Flaky .Net and server tests [tinkerpop]

Posted by "Cole-Greer (via GitHub)" <gi...@apache.org>.
Cole-Greer merged PR #3431:
URL: https://github.com/apache/tinkerpop/pull/3431


-- 
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: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org