-
Notifications
You must be signed in to change notification settings - Fork 472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
: failed #3595
Comments
Looks like this one is a 20m timeout in one run of the crossversion test, except we can't see anything because the artifacts couldn't be uploaded due to an OOM. Possibly just a repro of #3594. |
. failed with artifacts on refs/heads/master @ 98cadc9f9bcd:
Help
|
. failed with artifacts on refs/heads/master @ 98cadc9f9bcd:
|
. failed with artifacts on refs/heads/master @ 4301b8f511d9:
Help
|
. failed with artifacts on refs/heads/master @ 4301b8f511d9:
|
Before the recent compaction cleanup, the output table writer was initialized lazily, which means that until the first key was written the estimated size was 0. So even with a tiny target size, we will get at least one point key and any associated spans. Now the table writer is initialized up front and the estimated size of an empty table writer is 8. When the target file size is less than that, we try to split the table as soon as possible (and emit just a small piece of a span in many cases). These many tiny tables slow down the tests a lot, to the point of timing out certain operations. This change sets the minimum target file size in the metamorphic tests to 12. Fixes cockroachdb#3594 Fixes cockroachdb#3595
. failed with artifacts on refs/heads/master @ f03e7efeebfd:
Help
See also: How To Investigate a Go Test Failure (internal)
Same failure on other branches
This test on roachdash | Improve this report!
The text was updated successfully, but these errors were encountered: