jedcunningham commented on a change in pull request #19010:
URL: https://github.com/apache/airflow/pull/19010#discussion_r731283680



##########
File path: Dockerfile.ci
##########
@@ -400,6 +400,16 @@ LABEL org.apache.airflow.distro="debian" \
   org.opencontainers.image.title="Continuous Integration Airflow Image" \
   org.opencontainers.image.description="Installed Apache Airflow with 
Continuous Integration dependencies"
 
+# This one is to workaround https://github.com/apache/airflow/issues/17546
+# issue with /usr/lib/x86_64-linux-gnu/libstdc++.so.6: cannot allocate memory 
in static TLS block
+# We do not yet a more "correct" solution to the problem but in order to avoid 
raising new issues
+# by users of the prod image, we implement the workaround now.
+# The side effect of this is slightly (in the range of 100s of milliseconds) 
slower load for any
+# binary started and a little memory used for Heap allocated by initialization 
of libstdc++
+# This overhead is not happening for binaries that already link dynamically 
libstdc++
+ENV LD_PRELOAD="/usr/lib/x86_64-linux-gnu/libstdc++.so.6"

Review comment:
       It apparently has been happening on CI too, check out #18745.




-- 
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-unsubscr...@airflow.apache.org

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


Reply via email to