Change `CohereHook.get_conn` type from property to method in the next major `apache-airflow-providers-cohere`

[Taragolis]

Body

CohereHook change use property

airflow/airflow/providers/cohere/hooks/cohere.py

Lines 55 to 56 in 9f9ae33

	@cached_property
	def get_conn(self) -> cohere.Client:

Instead of method

airflow/airflow/hooks/base.py

Lines 98 to 100 in 9f9ae33

	def get_conn(self) -> Any:
	"""Return connection for the hook."""
	raise NotImplementedError()

Should be changed into the next major version.

For save caching it might be changed to something like (just for a reference)

class CohereHook(BaseHook):
    ...
    def get_conn(self) -> client.Cohere: 
    """Some useful docstring""" 
        return self.client
    
    @cached_property
    def get_conn(self) -> client.Cohere: 
    """Some useful docstring""" 
        ... # Some implementation for obtain client here

Committer

• I acknowledge that I am a maintainer/committer of the Apache Airflow project.

[eladkal]

@okirialbert following #45267 can you look into this one as well?

[sjyangkevin]

Hi, I found this from the new contributors channel, and I am interested in working on it. Thanks.

[sjyangkevin]

Hi,

I have raised a PR for this issue #50470. Let me know if there is any feedback or things to change. Thank you for your time to review in advance.