xchwan commented on issue #47950: URL: https://github.com/apache/airflow/issues/47950#issuecomment-3310815386
> > Does _get_aad_token not have this limit? > > Hi you can see the implementation here https://github.com/apache/airflow/blob/e66b237238c6eef1826c001e584a1992b3d879e7/providers/databricks/src/airflow/providers/databricks/hooks/databricks_base.py#L297C9-L297C23 it caches the auth token so it does not instantiate ManagedIdentityCredential. > > [@xchwan](https://github.com/xchwan) I can't test your change. I also add cache for the check, you can find ```if self._metadata_cache and time.time() < self._metadata_expiry:``` in PR. You said it cache can work means it already get info. So I guess it possible to handle. It is ok that you can't test. I just want to know you opinion. -- 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]
