Skip to content
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.

Sign up for GitHub

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

Jump to bottom

crosscluster/logical: TestFourWayReplication failed #139673

Closed
cockroach-teamcity opened this issue Jan 23, 2025 · 0 comments
Closed

crosscluster/logical: TestFourWayReplication failed #139673

cockroach-teamcity opened this issue Jan 23, 2025 · 0 comments
Assignees
@msbutler
Labels
A-disaster-recovery branch-release-25.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. P-3 Issues/test failures with no fix SLA T-disaster-recovery

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Jan 23, 2025 •
edited by cockroach-jira-scripts
Loading

crosscluster/logical.TestFourWayReplication failed on release-25.1 @ 0fb79d22b0fc0719f28a1e3993be2c4babf5155e:

Fatal error:

panic: test timed out after 5m0s
running tests:
	TestFourWayReplication (4m2s)

Stack:

goroutine 236019 [running]:
testing.(*M).startAlarm.func1()
	GOROOT/src/testing/testing.go:2366 +0x385
created by time.goFunc
	GOROOT/src/time/sleep.go:177 +0x2d
Log preceding fatal error 

=== RUN   TestFourWayReplication
    test_log_scope.go:165: test logs captured to: outputs.zip/logTestFourWayReplication2415387372
    test_log_scope.go:76: use -show-logs to present logs inline

Parameters:

  • attempt=1
  • run=4
  • shard=1
Help

See also: How To Investigate a Go Test Failure (internal)

/cc @cockroachdb/disaster-recovery

This test on roachdash | Improve this report!

Jira issue: CRDB-46767
@cockroach-teamcity cockroach-teamcity added branch-release-25.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-disaster-recovery labels Jan 23, 2025
@exalate-issue-sync exalate-issue-sync bot added the P-3 Issues/test failures with no fix SLA label Jan 27, 2025
@msbutler msbutler removed the release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. label Jan 27, 2025
msbutler added a commit to msbutler/cockroach that referenced this issue Jan 28, 2025
@msbutler
crosscluster/logical: use large test pool
4d4e767 
This package spins up several TestServers. We've also seen many tests timeout,
likely due to resource exhaustion.

Informs cockroachdb#138277
Informs cockroachdb#139673

Release note: none
msbutler added a commit to msbutler/cockroach that referenced this issue Jan 28, 2025
@msbutler
crosscluster/logical: use large test pool
31e48d8 
This package spins up several TestServers. We've also seen many tests timeout,
likely due to resource exhaustion.

Informs cockroachdb#138277
Informs cockroachdb#139673

Release note: none
@msbutler msbutler mentioned this issue Jan 28, 2025
Merged
msbutler added a commit to msbutler/cockroach that referenced this issue Jan 28, 2025
@msbutler
crosscluster/logical: use large test pool
0d07ed8 
This package spins up several TestServers. We've also seen many tests timeout,
likely due to resource exhaustion.

Informs cockroachdb#138277
Informs cockroachdb#139673

Release note: none
craig bot pushed a commit that referenced this issue Jan 31, 2025
@annrpom @msbutler
@rytaft @mgartner @andrewbaptist
Merge #139256 #139955 #140057 #140065 #140203
281a25b 
139256: sql/rowenc: reduce index key prefix calls r=annrpom a=annrpom

### sql/rowenc: reduce index key prefix calls
This patch removes redundant calls to `MakeIndexKeyPrefix` during
the construction of `IndexEntry`s by saving each first-time call in a
map that we can later lookup. Previously, we would make this call
for each row; however, as the prefix (table id + index id) for a
particular index remains the same, we do not need to do any
recomputation.

Epic: [CRDB-42901](https://cockroachlabs.atlassian.net/browse/CRDB-42901)
Fixes: #137798

Release note: None

---

### sql/rowexec: run BenchmarkIndexBackfill on-disk
Epic: none
Release note: None


139955: crosscluster/logical: use large test pool r=rickystewart a=msbutler

This package spins up several TestServers. We've also seen many tests timeout, likely due to resource exhaustion.

Informs #138277
Informs #139673

Release note: none

140057: sqlsmith: randomly generate auto partial stats in sqlsmith tests r=rytaft a=rytaft

Epic: None

Release note: None

140065: opt: add `optimizer_min_row_count` session setting r=mgartner a=mgartner

Informs #64570
Informs #130201

Release note (sql change): The `optimizer_min_row_count` session setting
has been added which sets a lower bound on row count estimates for
relational expressions during query planning. A value of zero, which is
the default, indicates no lower bound. Note that if this is set to a
value greater than zero, a row count of zero can still be estimated for
expressions with a cardinality of zero, e.g., for a contradictory
filter. Setting this to a value higher than 0, such as 1, may yield
better query plans in some cases, such as when statistics are frequently
stale and inaccurate.


140203: server: split join_list to its own file r=RaduBerinde a=andrewbaptist

Previously StoreSpec and JoinListType were in the same file. This commit moves JoinListType and its test to a separate file.

Epic: CRDB-41111

Release note: None

Co-authored-by: Annie Pompa <[email protected]>
Co-authored-by: Michael Butler <[email protected]>
Co-authored-by: Rebecca Taft <[email protected]>
Co-authored-by: Marcus Gartner <[email protected]>
Co-authored-by: Andrew Baptist <[email protected]>
blathers-crl bot pushed a commit that referenced this issue Jan 31, 2025
@msbutler
crosscluster/logical: use large test pool
d779c7f 
This package spins up several TestServers. We've also seen many tests timeout,
likely due to resource exhaustion.

Informs #138277
Informs #139673

Release note: none
@blathers-crl blathers-crl bot mentioned this issue Jan 31, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msbutler msbutler

Labels
A-disaster-recovery branch-release-25.1 C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. P-3 Issues/test failures with no fix SLA T-disaster-recovery
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cockroach-teamcity @msbutler