Taragolis commented on code in PR #28625:
URL: https://github.com/apache/airflow/pull/28625#discussion_r1058459895


##########
docs/apache-airflow-providers-microsoft-mssql/index.rst:
##########
@@ -91,6 +91,8 @@ PIP package                              Version required
 ``pymssql``                              ``>=2.1.5; platform_machine != 
"aarch64"``
 =======================================  
==========================================
 
+Note: For ``pymssql`` installation some Linux distributions ``pip`` version is 
too old to support all the flavors of ``manylinux`` wheels, so upgrading 
``pip`` is necessary. An example of such distributions would be Ubuntu 18.04 or 
Python3.6 module in RHEL8 and CentOS8.

Review Comment:
   I just think that it could be general advice not to use system `pip` as well 
as outdated `pip` in 
https://airflow.apache.org/docs/apache-airflow/stable/installation/installing-from-pypi.html
   
   We do not have any official restrictions for `pip` version however in 
official Airflow Docker images uses latest version of pip (at the moment when 
new Airflow version released).
   
   But I'm sure that users might have a low of issues if the tried to install 
airflow by use default system pip:
   Ubuntu 18.04: python3-pip 9.0.1
   Ubuntu 20.04: python3-pip 20.0.2
   Debian 10: python3-pip 18.1
   Debian 11: python3-pip 20.3.4
   Centos 7: python3-pip 9.0.3
   Centos 8: python3-pip 9.0.3
   



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