potiuk commented on PR #30704: URL: https://github.com/apache/airflow/pull/30704#issuecomment-1523867035
> > This functionality is basically `functools.lru_cache`. > > yes and no because this is an in-function cache. `functools.lru_cache` is intended to provide a cache that is persisted at the object level I think ? Not realy. @uranusjr is right. lru_cache caches the value of function keyed by arguments of the function (the only condition is that the arguments must be hashable). So it does precisely what the extra (unneded) code does in this case.. -- 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]
