Skip to content

Commit 433f7da

Browse files
authored
Error handling semantics
1 parent d5bebe4 commit 433f7da

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

servicex_analysis_utils/materialization.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ def to_awk(deliver_dict, dask=False, iterator=False, **kwargs):
5454

5555
for sample, paths in deliver_dict.items():
5656
if not paths:
57-
raise ValueError(f"ServiceX result path list for {sample} cannot be empty.")
57+
raise RuntimeError(f"Delivered result file path list for {sample} is empty.")
5858
# Check file type
5959
f_type = str(paths[0])
6060
if ".root" in f_type:
@@ -63,8 +63,8 @@ def to_awk(deliver_dict, dask=False, iterator=False, **kwargs):
6363
is_root = False
6464
# ServiceX supports only root/parquet in transformed files
6565
else:
66-
raise ValueError(
67-
f"Unsupported file format: '{paths[0]}'. Files must be ROOT (.root) or Parquet (.parquet, .pq)"
66+
raise RuntimeError(
67+
f"Unsupported file format delivered by ServiceX: '{paths[0]}'. Files must be ROOT (.root) or Parquet (.parquet, .pq)"
6868
)
6969

7070
try:

0 commit comments

Comments
 (0)