Skip to content

Commit

Permalink
[external data] remove LegacyNamedTupleMixin (#23447)
Browse files Browse the repository at this point in the history
follow up to dagster-io/internal#10910

## How I Tested These Changes

bk
  • Loading branch information
alangenfeld authored Aug 15, 2024
1 parent d581365 commit b6f91e1
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@
from dagster._core.storage.io_manager import IOManagerDefinition
from dagster._core.storage.tags import COMPUTE_KIND_TAG
from dagster._core.utils import is_valid_email
from dagster._record import IHaveNew, LegacyNamedTupleMixin, record, record_custom
from dagster._record import IHaveNew, record, record_custom
from dagster._serdes import whitelist_for_serdes
from dagster._serdes.serdes import FieldSerializer, is_whitelisted_for_serdes_object
from dagster._time import datetime_from_timestamp
Expand Down Expand Up @@ -267,7 +267,7 @@ def __new__(
old_fields={"is_job": True},
)
@record
class ExternalJobData(LegacyNamedTupleMixin):
class ExternalJobData:
name: str
job_snapshot: JobSnapshot
active_presets: Sequence[ExternalPresetData]
Expand All @@ -279,7 +279,7 @@ class ExternalJobData(LegacyNamedTupleMixin):
storage_field_names={"external_job_data": "external_pipeline_data"},
)
@record
class ExternalJobSubsetResult(LegacyNamedTupleMixin):
class ExternalJobSubsetResult:
success: bool
error: Optional[SerializableErrorInfo] = None
external_job_data: Optional[ExternalJobData] = None
Expand Down

0 comments on commit b6f91e1

Please sign in to comment.