[GitHub] [airflow] uranusjr edited a comment on issue #18116: Methods `TaskInstance.get_previous_execution_date()` & `TaskInstance.get_previous_ti()` are lying !

[GitBox <gi...@apache.org>]

uranusjr edited a comment on issue #18116:
URL: https://github.com/apache/airflow/issues/18116#issuecomment-916140849


   There was another issue on this recently but I couldn’t find it right now. The bottom line is this method is like this since, like, forever, and we can’t just change it now.
   
   From what I can tell, `get_previous_ti(state)` is _never_ intended to be used as a public API (only `get_previous_ti()` withut arguments is, which gives you the previous TaskInstance as the name suggests), so perhaps we should separate the two signatures (and rename the with-state version to something better).


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org