Skip to content

Commit

Permalink
fixup! Issue #656: make sure CsvJobDatabase uses expected dtypes
Browse files Browse the repository at this point in the history
  • Loading branch information
soxofaan committed Nov 13, 2024
1 parent 89f6fe1 commit f5eb8a3
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions openeo/extra/job_management.py
Original file line number Diff line number Diff line change
Expand Up @@ -181,13 +181,12 @@ def start_job(
"""

# Expected columns in the job DB dataframes.
# Mapping of column name to (dtype, default value)
# TODO: make this part of public API when settled?
_COLUMN_REQUIREMENTS: Mapping[str, _ColumnProperties] = {
"id": _ColumnProperties(dtype="str"),
"backend_name": _ColumnProperties(dtype="str"),
"status": _ColumnProperties(dtype="str", default="not_started"),
# TODO: use proper date/time dtype instead of leagacy str for start times?
# TODO: use proper date/time dtype instead of legacy str for start times?
"start_time": _ColumnProperties(dtype="str"),
"running_start_time": _ColumnProperties(dtype="str"),
# TODO: these columns "cpu", "memory", "duration" are not referenced explicitly from MultiBackendJobManager,
Expand Down

0 comments on commit f5eb8a3

Please sign in to comment.