add some useful comments

GoogleCloudPlatform · Oct 11, 2024 · 78906ea · 78906ea
1 parent 9f3397e
commit 78906ea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/perfmetrics/scripts/testing_on_gke/examples/dlio/parse_logs.py b/perfmetrics/scripts/testing_on_gke/examples/dlio/parse_logs.py
@@ -317,6 +317,8 @@ def writeOutput(
               f"Error while creating new output row for key={key},"
               f" scenario={scenario}, epoch={i}, r={r}: {e}"
           )
+          # Put "ERROR" in each field in the row to indicate that this datsa-row
+          # could not be parsed.
           rows.append((("ERROR",) * len(_HEADER)))
 
   export_to_csv(output_file_path=args.output_file, header=_HEADER, rows=rows)

diff --git a/perfmetrics/scripts/testing_on_gke/examples/fio/parse_logs.py b/perfmetrics/scripts/testing_on_gke/examples/fio/parse_logs.py
@@ -351,6 +351,8 @@ def writeOutput(
               f"Error while creating new output row for key={key},"
               f" scenario={scenario}, epoch={i}, r={r}: {e}"
           )
+          # Put "ERROR" in each field in the row to indicate that this datsa-row
+          # could not be parsed.
           rows.append((("ERROR",) * len(_HEADER)))
 
   export_to_csv(output_file_path=args.output_file, header=_HEADER, rows=rows)