potiuk commented on code in PR #33355:
URL: https://github.com/apache/airflow/pull/33355#discussion_r1324017566


##########
airflow/operators/python.py:
##########
@@ -606,7 +614,60 @@ def _prepare_venv(self, venv_path: Path) -> None:
             index_urls=self.index_urls,
         )
 
+    def _calculate_cache_hash(self) -> str:

Review Comment:
   Not the scheduler but the worker.. I think it's perfectly plausible if the 
two tasks start at different moment (one before the change and one after). Why 
do you think it's not possible - it's enought that one LocalTaskJob starts (on 
the same Celery or LocalExecutor worker) just before the "rich" change is done 
and the other is started "just after".  The first one will run 
PythonVirtualOperator with "rich==2.29" parameter, and the other with 
"rich==2.30".
   
   Why do you think its not possible?



-- 
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]

Reply via email to