deephaven / deephaven-core

Deephaven Community Core
Other
257 stars 80 forks source link

fix: Allows clients to expect that query performance results have been logged before they receive a response #6334

Closed rcaudy closed 2 weeks ago

rcaudy commented 2 weeks ago

Defer response messaging to onSuccess for all ExportBuilders that use QueryPerformanceRecorder. This allows clients to expect that query performance results have been logged before they receive a response.

Change summary:

nbauernfeind commented 2 weeks ago

LGTM, waiting for a ping from @niloc132 before approving in case he wants a final look before the merge.