Skip to content

Commit 150d59b

Browse files
amoghrajeshGitOps Bot
authored andcommitted
Remove some irrelevant TODOs in task sdk (apache#56506)
1 parent 9ba95dd commit 150d59b

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

task-sdk/src/airflow/sdk/definitions/variable.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ class Variable:
4444
value: Any | None = None
4545
description: str | None = None
4646

47-
# TODO: Extend this definition for reading/writing variables without context
4847
@classmethod
4948
def get(cls, key: str, default: Any = NOTSET, deserialize_json: bool = False):
5049
from airflow.sdk.exceptions import AirflowRuntimeError, ErrorType

task-sdk/src/airflow/sdk/execution_time/context.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -207,9 +207,6 @@ async def _async_get_connection(conn_id: str) -> Connection:
207207
from airflow.sdk.execution_time.supervisor import ensure_secrets_backend_loaded
208208
from airflow.sdk.execution_time.task_runner import SUPERVISOR_COMMS
209209

210-
# TODO: check cache first
211-
# enabled only if SecretCache.init() has been called first
212-
213210
# Try secrets backends first using async wrapper
214211
backends = ensure_secrets_backend_loaded()
215212
for secrets_backend in backends:

0 commit comments

Comments
 (0)