mik-laj commented on a change in pull request #16170:
URL: https://github.com/apache/airflow/pull/16170#discussion_r642137352



##########
File path: scripts/in_container/prod/entrypoint_prod.sh
##########
@@ -311,6 +311,10 @@ if [[ -n "${_AIRFLOW_WWW_USER_CREATE=}" ]] ; then
     create_www_user
 fi
 
+if [[ -n "${_PIP_ADDITIONAL_REQUIREMENTS=}" ]] ; then
+    pip install --no-cache-dir --user "${_PIP_ADDITIONAL_REQUIREMENTS=}"

Review comment:
       I wonder if it is worth adding a warning here that this option has a 
drastic impact on performance and recommend migrating to the standard method. 
It will be most painful if the user is to use the `airflow.sh` script as it has 
been optimized for speed and this option will have a significant impact on it.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to