Lee-W commented on issue #54714:
URL: https://github.com/apache/airflow/issues/54714#issuecomment-3220353438
Hey @amoghrajesh , I tried to list down a few things we already have with a
few questions.
Also, AIR311 and AIR312 are the ones that suggest users to update, but won't
break. Will the user still be able to use things like
`airflow.sensors.date_time.DateTimeSensor`? If so, I think we'll need to create
a new error code (e.g., `AIR313` required to update after Airflow 3.1)
For the AIR301, we don't need to do anything. They won't work for 3.0 anyway.
There are some missing ones, probably due to missing newsfragments back to
the time we list down the rules.
---
### airflow.utils module
- [ ] `airflow.utils.timezone.*` → `airflow.sdk.timezone` **(would be better
to actaully list down everything)**
### airflow.sensors module
- [ ] `airflow.sensors.base.BaseSensorOperator` →
`airflow.sdk.bases.sensor.BaseSensorOperator` **(AIR301)**
- [ ] `airflow.sensors.date_time.DateTimeSensor` →
`airflow.providers.standard.sensors.date_time.DateTimeSensor` *(AIR312)*
- [ ] `airflow.sensors.date_time.DateTimeSensorAsync` →
`airflow.providers.standard.sensors.date_time.DateTimeSensorAsync` *(AIR312)*
### airflow.secrets module
- [ ] `airflow.secrets.cache.SecretCache` →
`airflow.sdk.execution_time.cache.SecretCache` **(AIR301) (should it be
`airflow.sdk.SecretCache` instead?)**
### airflow.notifications module
- [ ] `airflow.notifications.basenotifier.BaseNotifier` →
`airflow.sdk.bases.notifier.BaseNotifier` **(AIR301)**
### airflow.models module
- [ ] `airflow.models.abstractoperator.AbstractOperator` →
`airflow.sdk.definitions._internal.abstractoperator.AbstractOperator` (should
these internal still be used?)
- [ ] `airflow.models.abstractoperator.NotMapped` →
`airflow.sdk.definitions._internal.abstractoperator.NotMapped`
- [ ] `airflow.models.baseoperator.chain` →
`airflow.sdk.bases.operator.chain` **(AIR301) (should it be `airflow.sdk.chain`
instead)**
### airflow.macros module
- [ ] `airflow.macros.*` → `airflow.sdk.execution_time.macros` **(would be
better to actually list down everything)**
### airflow.io module
- [ ] `airflow.io.attach` → `airflow.sdk.io.attach` (do you mean
`airflow.io.store.attach`?) *(AIR312)*
- [ ] `airflow.io.path.ObjectStoragePath` → `airflow.sdk.ObjectStoragePath`
*(AIR312)*
### airflow.hooks module
- [ ] `airflow.hooks.filesystem.FSHook` →
`airflow.providers.standard.hooks.filesystem.FSHook` (AIR311)
### airflow.decorators module
- [ ] `airflow.decorators.dag` → `airflow.sdk.dag` *(AIR311)*
- [ ] `airflow.decorators.task` → `airflow.sdk.task` *(AIR311)*
- [ ] `airflow.decorators.task_group` → `airflow.sdk.task_group` *(AIR311)*
---
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
