Skip to content

Commit c430a90

Browse files
committed
fixed some paths in gymlib integtests
1 parent e4235fe commit c430a90

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

env/tests/_set_up_gymlib_integtest_workspace.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ set -euo pipefail
1616
# INTENDED_DBDATA_HARDWARE can be set elsewhere (e.g. by tests_ci.yaml) but we use hdd by default.
1717
INTENDED_DBDATA_HARDWARE="${INTENDED_DBDATA_HARDWARE:-hdd}"
1818

19-
python3 task.py benchmark $BENCHMARK data $SCALE_FACTOR
19+
python3 task.py benchmark $BENCHMARK tables $SCALE_FACTOR
2020
python3 task.py benchmark $BENCHMARK workload --scale-factor $SCALE_FACTOR
2121

2222
python3 task.py dbms postgres build

env/tests/gymlib_integtest_util.py

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from typing import Optional
55

66
# TODO: remove symlinks_paths from the import
7-
from gymlib.symlinks_paths import get_workload_dirname, get_workload_suffix
7+
from gymlib.symlinks_paths import get_workload_suffix, get_workload_symlink_path
88

99
from benchmark.tpch.constants import DEFAULT_TPCH_SEED
1010
from env.tuning_artifacts import TuningMetadata
@@ -14,7 +14,6 @@
1414
get_default_dbdata_parent_dpath,
1515
get_default_pgbin_path,
1616
get_default_pristine_dbdata_snapshot_path,
17-
get_default_workload_path,
1817
get_workspace_path_from_config,
1918
)
2019

@@ -88,14 +87,11 @@ def get_default_metadata() -> TuningMetadata:
8887
)
8988
return TuningMetadata(
9089
workload_path=fully_resolve_path(
91-
get_default_workload_path(
90+
get_workload_symlink_path(
9291
dbgym_workspace.dbgym_workspace_path,
9392
GymlibIntegtestManager.BENCHMARK,
94-
get_workload_dirname(
95-
GymlibIntegtestManager.BENCHMARK,
96-
GymlibIntegtestManager.SCALE_FACTOR,
97-
suffix,
98-
),
93+
GymlibIntegtestManager.SCALE_FACTOR,
94+
suffix
9995
),
10096
),
10197
pristine_dbdata_snapshot_path=fully_resolve_path(

0 commit comments

Comments
 (0)