This is an automated email from the ASF dual-hosted git repository.
uranusjr pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 9f17caf6319 Remove no-longer-used HIDE_ATTRS_FROM_UI (#64560)
9f17caf6319 is described below
commit 9f17caf6319317aa83ec3467640a332b04ad27a9
Author: Tzu-ping Chung <[email protected]>
AuthorDate: Wed Apr 1 10:38:46 2026 +0800
Remove no-longer-used HIDE_ATTRS_FROM_UI (#64560)
---
.../prek/check_base_operator_partial_arguments.py | 2 --
.../sdk/definitions/_internal/abstractoperator.py | 25 +---------------------
.../src/airflow/sdk/definitions/mappedoperator.py | 6 +-----
3 files changed, 2 insertions(+), 31 deletions(-)
diff --git a/scripts/ci/prek/check_base_operator_partial_arguments.py
b/scripts/ci/prek/check_base_operator_partial_arguments.py
index ee2c7639f08..67ef666e310 100755
--- a/scripts/ci/prek/check_base_operator_partial_arguments.py
+++ b/scripts/ci/prek/check_base_operator_partial_arguments.py
@@ -49,8 +49,6 @@ IGNORED = {
"default_args",
# Deprecated and is aliased to max_active_tis_per_dag.
"task_concurrency",
- # attrs internals.
- "HIDE_ATTRS_FROM_UI",
# Only on BaseOperator.
"_dag",
"output",
diff --git a/task-sdk/src/airflow/sdk/definitions/_internal/abstractoperator.py
b/task-sdk/src/airflow/sdk/definitions/_internal/abstractoperator.py
index 00b811146a6..152efdecc40 100644
--- a/task-sdk/src/airflow/sdk/definitions/_internal/abstractoperator.py
+++ b/task-sdk/src/airflow/sdk/definitions/_internal/abstractoperator.py
@@ -26,7 +26,7 @@ from collections.abc import (
Iterable,
Iterator,
)
-from typing import TYPE_CHECKING, Any, ClassVar, TypeAlias
+from typing import TYPE_CHECKING, Any, TypeAlias
from airflow.sdk import TriggerRule, WeightRule
from airflow.sdk.configuration import conf
@@ -114,29 +114,6 @@ class AbstractOperator(Templater, DAGNode):
is_setup: bool = False
is_teardown: bool = False
- HIDE_ATTRS_FROM_UI: ClassVar[frozenset[str]] = frozenset(
- (
- "log",
- "dag", # We show dag_id, don't need to show this too
- "node_id", # Duplicates task_id
- "task_group", # Doesn't have a useful repr, no point showing in UI
- "inherits_from_empty_operator", # impl detail
- "inherits_from_skipmixin", # impl detail
- # Decide whether to start task execution from triggerer
- "start_trigger_args",
- "start_from_trigger",
- # For compatibility with TG, for operators these are just the
current task, no point showing
- "roots",
- "leaves",
- # These lists are already shown via *_task_ids
- "upstream_list",
- "downstream_list",
- # Not useful, implementation detail, already shown elsewhere
- "global_operator_extra_link_dict",
- "operator_extra_link_dict",
- )
- )
-
@property
def is_async(self) -> bool:
return False
diff --git a/task-sdk/src/airflow/sdk/definitions/mappedoperator.py
b/task-sdk/src/airflow/sdk/definitions/mappedoperator.py
index 7c0540421d4..94b5a9bfab9 100644
--- a/task-sdk/src/airflow/sdk/definitions/mappedoperator.py
+++ b/task-sdk/src/airflow/sdk/definitions/mappedoperator.py
@@ -21,7 +21,7 @@ import contextlib
import copy
import warnings
from collections.abc import Collection, Iterable, Iterator, Mapping, Sequence
-from typing import TYPE_CHECKING, Any, ClassVar, Literal, TypeGuard
+from typing import TYPE_CHECKING, Any, Literal, TypeGuard
import attrs
import methodtools
@@ -335,10 +335,6 @@ class MappedOperator(AbstractOperator):
This should be a name to call ``getattr()`` on.
"""
- HIDE_ATTRS_FROM_UI: ClassVar[frozenset[str]] =
AbstractOperator.HIDE_ATTRS_FROM_UI | frozenset(
- ("parse_time_mapped_ti_count", "operator_class", "start_trigger_args",
"start_from_trigger")
- )
-
def __hash__(self):
return id(self)
