We need also update
https://github.com/apache/airflow#release-process-for-providers
With new date

We need also update https://github.com/apache/airflow#release-process-for-providers With new date

Right. Updated.

I'd love @uranusjr if you take a look as well. The biggest part of pre-2.3 was removal of dttm from "XCom.get_value" - mostly in operator links.

Also I am not sure if what I've done is correct. I assumed that after migration to 2.3 each task instance will have a "task_instance_key" and we will be able to use task instance key and get_value even for task instances created in previous Airflow versions (I believe migration takes care about it).

I would love your comments here @uranusjr if I understood correctly that we should genereally skip all the dttm usage and rely on task_instance_key being always present.

I have found however one case where it is not as straightforward and dttm is still used and I am not sure if we want to do anything about it:

class ExternalDagLink(BaseOperatorLink):
    """
    Operator link for ExternalTaskSensor and ExternalTaskMarker.
    It allows users to access DAG waited with ExternalTaskSensor or cleared by ExternalTaskMarker.
    """

    name = 'External DAG'

    def get_link(self, operator, dttm):
        ti = TaskInstance(task=operator, execution_date=dttm)
        operator.render_template_fields(ti.get_template_context())
        query = {"dag_id": operator.external_dag_id, "execution_date": dttm.isoformat()}
        return build_airflow_url_with_query(query)

In all the other cases I converted 'get_link" to

    def get_link(
        self,
        operator: BaseOperator,
        *,
        ti_key: TaskInstanceKey,

We also have:

We also have:

airflow/airflow/providers/oracle/hooks/oracle.py

Lines 428 to 441 in b254a9f

	# TODO: Merge this implementation back to DbApiHook when dropping
	# support for Airflow 2.2.
	def test_connection(self):
	"""Tests the connection by executing a select 1 from dual query"""
	status, message = False, ''
	try:
	if self.get_first("select 1 from dual"):
	status = True
	message = 'Connection successfully tested'
	except Exception as e:
	status = False
	message = str(e)
	return status, message

Ah cool. Indeed.

Actually, the DBApi one we could do before already after we split-out common.sql.

Fixed.

(also done the same for Trino, Snowflake and Databricks)

Ahh we can also handle:

This one also should cut the "compatibility" test by more than hour. Currently just installing latest providers on airflow 2.2 takes > 1 hour because pip tries to resolve dependencies and it tries very hard to do so (this is because of the complex dependency resolution caused by "old" airflow and "new" providers). This is -yet another reason why the 12 months compatibility for providers is a good idea.

All right - seems I get it alll green - @uranusjr - is it possible you take a look at the XCom.get_value() changes before I merge this one?

Just did, I think you did it correctly.

Deferrable Operators only work starting Airflow 2.2

Indeed. Thanks @eladkal for all the extra checks :)