-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
sql/importer: TestImportDefaultWithResume failed [ingestKVs hangs with BulkAdderFlushesEveryBatch] #92910
Comments
happened here too |
This new failure looks to be the same as #91828 (comment) The cockroach/pkg/sql/importer/import_processor.go Lines 544 to 548 in a16106e
@adityamaru or @dt - do you have any idea why this might happen? |
Let me take a look, not sure off the top of my head. |
Since it seems related to the BulkAdder behavior, I'm putting this in the DR team board. Let us know if you want to give it back. |
cc @cockroachdb/disaster-recovery |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ ca079ac1fc8a8f7a5360f5b1f4fa210c14dc8410:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 859e1e1f26d79af43cce16519f9ba984baaa495c:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 8a5cb5109d6b7e122e996a44f940bfd61698cd81:
Parameters: |
I'll start a bisect today to figure out what started causing this to fail. |
The failure started after #89649, now trying to look into the patch but from the description the failure we're saying makes sense. |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 1b4947bf4e1036a7c55996d2cb6ab815254a3c8c:
Parameters: |
I have a patch that fixes the flake that looks very similar to the bigger change @stevendanna has around import processor shutdown in #91615. We plan on merging this soon, so I'm going to hold off on sending an independent patch. |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ ec095bc2fdbe4e518b076db20e4920fab67222bf:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 3910ac409195ab6ab2ec3e9f6eda553048afa23f:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ f2b00e8039af6ea8887ec124dad8daf19da6fbf1:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ c050c9b4b57ecf2ceb5d449c31c617fe12c920e0:
Parameters: |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 942a4d468e9c8ad0ef45a7be33f0a326dfb19fef:
Parameters: |
Should be closed by #93782 |
sql/importer.TestImportDefaultWithResume failed with artifacts on master @ 7cb778506d75bbef2eb90abccaa75b9dc7e3fb91:
Parameters:
TAGS=bazel,gss
Help
See also: How To Investigate a Go Test Failure (internal)
This test on roachdash | Improve this report!
Jira issue: CRDB-22045
The text was updated successfully, but these errors were encountered: