Skip to content

Commit acb9e94

Browse files
committed
Add TODO to support more levels of nesting in the future
Signed-off-by: Finn Roblin <finnrobl@amazon.com>
1 parent 26b0d8d commit acb9e94

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

osbenchmark/workload/params.py

+4-1
Original file line numberDiff line numberDiff line change
@@ -983,9 +983,12 @@ def partition(self, partition_index, total_partitions):
983983

984984
def get_split_fields(self) -> Tuple[str, str]:
985985
fields_as_array = self.field_name.split(self.NESTED_FIELD_SEPARATOR)
986+
987+
# TODO: Add support to multiple levels of nesting if a future benchmark requires it.
988+
986989
if len(fields_as_array) != 2:
987990
raise ValueError(
988-
f"Field name {self.field_name} is not a nested field name"
991+
f"Field name {self.field_name} is not a nested field name. Currently we support only 1 level of nesting."
989992
)
990993
return fields_as_array[0], fields_as_array[1]
991994

0 commit comments

Comments
 (0)