Skip to content

Commit

Permalink
roachtest: reenable skipped import tests
Browse files Browse the repository at this point in the history
With cockroachdb#31875 in, import/tpch/nodes=4 has been passing reliably, and the
expectation is that the nodes=8,32 versions will follow suit (at least
they passed just now).

Release note: None
  • Loading branch information
tbg committed Nov 30, 2018
1 parent 5b6eeb1 commit 5ceea1e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
7 changes: 3 additions & 4 deletions pkg/cmd/roachtest/import.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,9 @@ func registerImportTPCH(r *registry) {
nodes int
timeout time.Duration
}{
{4, 6 * time.Hour},
// TODO(peter,tschottdorf): re-enable
// {8, 4 * time.Hour},
// {32, 3 * time.Hour},
{4, 6 * time.Hour}, // typically 4-5h
{8, 4 * time.Hour}, // typically 3h
{32, 3 * time.Hour},
} {
r.Add(testSpec{
Name: fmt.Sprintf(`import/tpch/nodes=%d`, item.nodes),
Expand Down
3 changes: 1 addition & 2 deletions pkg/cmd/roachtest/restore.go
Original file line number Diff line number Diff line change
Expand Up @@ -218,8 +218,7 @@ func registerRestore(r *registry) {
timeout time.Duration
}{
{10, 6 * time.Hour},
// TODO(peter,tschottdorf): re-enable
// {32, 3 * time.Hour},
{32, 3 * time.Hour},
} {
r.Add(testSpec{
Name: fmt.Sprintf("restore2TB/nodes=%d", item.nodes),
Expand Down

0 comments on commit 5ceea1e

Please sign in to comment.