Skip to content

Loosen case in FrameFileWriterTest.#16938

Merged
clintropolis merged 1 commit intoapache:masterfrom
gianm:fix-ff-test-case
Aug 21, 2024
Merged

Loosen case in FrameFileWriterTest.#16938
clintropolis merged 1 commit intoapache:masterfrom
gianm:fix-ff-test-case

Conversation

@gianm
Copy link
Copy Markdown
Contributor

@gianm gianm commented Aug 21, 2024

The specific error on a truncated file can vary based on how the final frame of the truncated file is written. This patch loosens the check so it passes regardless of how the truncated file is written.

The specific error on a truncated file can vary based on how the final
frame of the truncated file is written. This patch loosens the check so
it passes regardless of how the truncated file is written.
@clintropolis
Copy link
Copy Markdown
Member

cool, ran into this trying to fix another flaky test, #16932

@clintropolis clintropolis merged commit 0900236 into apache:master Aug 21, 2024
@gianm gianm deleted the fix-ff-test-case branch August 22, 2024 20:33
hevansDev pushed a commit to hevansDev/druid that referenced this pull request Aug 29, 2024
The specific error on a truncated file can vary based on how the final
frame of the truncated file is written. This patch loosens the check so
it passes regardless of how the truncated file is written.
@kfaraz kfaraz added this to the 31.0.0 milestone Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants