search

deephaven / deephaven-core

Deephaven Community Core
Other
257 stars 80 forks source link

TableServiceAsyncTest timeout unit test failure #4798

Closed devinrsmith closed 1 year ago

devinrsmith commented 1 year ago

It appears that the following test is sometimes failing:

2023-11-08T18:52:13.9139040Z io.deephaven.client.impl.TableServiceAsyncTest > longChainAsyncExportOnlyLast FAILED
2023-11-08T18:52:13.9142554Z     java.util.concurrent.TimeoutException at TableServiceAsyncTest.java:79

It suggests that there is an underlying race (potentially Batch / ETCR or related), or client implementation bug; but it's possible the timeout is too short given CI resources, TBD.

devinrsmith commented 1 year ago

As seen in https://github.com/deephaven/deephaven-core/actions/runs/6802272060/job/18495015941

devinrsmith commented 1 year ago

I've done 8 CI runs and so far have been unable to reproduce this issue. After #4802 merges, if this issues triggers in CI again we should have much more details to go off of.

devinrsmith commented 1 year ago

I think #4802 fixes this - leaving open for a week to see if we here anymore cases.

devinrsmith commented 1 year ago

Updated comment to specifically make sure this is about the timeout nature of this issue.