Skip to content

Commit

Permalink
Merge #48503
Browse files Browse the repository at this point in the history
48503: smithcmp: rename auto to 201auto in tpch config files r=yuzefovich a=yuzefovich

This commit also removes `auto` option from `tpchvec_smithcmp.toml`
config file because that option has different names in 20.2 and 20.1,
and we run the test on both. The alternative is to introduce a separate
config file, but I don't think `auto` option gives us much more test
coverage than already present `off` and `on` do.

Fixes: #48484.

Release note: None

Co-authored-by: Yahor Yuzefovich <[email protected]>
  • Loading branch information
craig[bot] and yuzefovich committed May 7, 2020
2 parents 84aa481 + 99d3d13 commit bb20495
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 9 deletions.
7 changes: 0 additions & 7 deletions pkg/cmd/roachtest/tpchvec_smithcmp.toml
Original file line number Diff line number Diff line change
Expand Up @@ -511,13 +511,6 @@ initsql = """
set vectorize=off;
"""

[databases.vec-auto]
addr = "postgresql://root@localhost:26257/tpch?sslmode=disable"
allowmutations = true
initsql = """
set vectorize=auto;
"""

[databases.vec-on]
addr = "postgresql://root@localhost:26257/tpch?sslmode=disable"
allowmutations = true
Expand Down
4 changes: 2 additions & 2 deletions pkg/cmd/smithcmp/tpch.toml
Original file line number Diff line number Diff line change
Expand Up @@ -513,11 +513,11 @@ initsql = """
set vectorize=off;
"""

[databases.vec-auto]
[databases.vec-201auto]
addr = "postgresql://root@localhost:26257/tpch?sslmode=disable"
allowmutations = true
initsql = """
set vectorize=auto;
set vectorize=201auto;
"""

[databases.vec-on]
Expand Down

0 comments on commit bb20495

Please sign in to comment.