Skip to content

fix: Ensure that most TableListener/MergedListener Notifications are processed while ensuring liveness; improve timeout handling in Table.awaitUpdate #2273

fix: Ensure that most TableListener/MergedListener Notifications are processed while ensuring liveness; improve timeout handling in Table.awaitUpdate

fix: Ensure that most TableListener/MergedListener Notifications are processed while ensuring liveness; improve timeout handling in Table.awaitUpdate #2273

Triggered via pull request November 22, 2024 23:14
@rcaudyrcaudy
opened #6422
Status Failure
Total duration 13s
Artifacts

conventional-pr-check.yml

on: pull_request_target
pr-check
4s
pr-check
Fit to window
Zoom out
Zoom in

Annotations

3 errors
pr-check
No release type found in pull request title "Ensure that most TableListener/MergedListener Notifications are processed while ensuring liveness; improve timeout handling in Table.awaitUpdate". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/ Available types: - feat: A new feature - fix: A bug fix - docs: Documentation only changes - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc) - refactor: A code change that neither fixes a bug nor adds a feature - perf: A code change that improves performance - test: Adding missing tests or correcting existing tests - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm) - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs) - chore: Other changes that don't modify src or test files - revert: Reverts a previous commit
TestTimeSeriesFilter.testIncremental2: io/deephaven/engine/table/impl/select/TestTimeSeriesFilter#L201
io.deephaven.base.verify.AssertionFailure: Assertion failed: asserted queuedNotificationStep == notificationStep, instead queuedNotificationStep == 10, notificationStep == -1, currentStep == 13, MergedListener == MergedListener(1487965527).
TestTimeSeriesFilter.testIncremental: io/deephaven/engine/table/impl/select/TestTimeSeriesFilter#L167
io.deephaven.base.verify.AssertionFailure: Assertion failed: asserted queuedNotificationStep == notificationStep, instead queuedNotificationStep == 7, notificationStep == -1, currentStep == 11, MergedListener == MergedListener(1525633317).