[GitHub] [pulsar] tuteng edited a comment on issue #5432: Revert "[Issue 4956]Add default loader for latest pyyaml"

[GitBox]

tuteng edited a comment on issue #5432: Revert "[Issue 4956]Add default loader 
for latest pyyaml"
URL: https://github.com/apache/pulsar/pull/5432#issuecomment-544786813
 
 
   @jerrypeng My guess is that python-yaml is the package that caused this 
problem. instead of installing python-pip, I directly use the following command 
to package it in 
[Dockerfile](https://github.com/apache/pulsar/blob/master/docker/pulsar/Dockerfile).
 it can be successful and the integration test can also pass.
   
   ```
   RUN curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py
   RUN python2.7 get-pip.py
   RUN pip2.7 install kazoo pyyaml==5.1.2
   RUN python3.7 get-pip.py
   RUN pip3.7 install kazoo pyyaml==5.1.2
   ```
   
   ```
   root@c4b89a6b7021:/pulsar# pip2.7 list
   DEPRECATION: Python 2.7 will reach the end of its life on January 1st, 2020. 
Please upgrade your Python as Python 2.7 won't be maintained after that date. A 
future version of pip will drop support for Python 2.7. More details about 
Python 2 support in pip, can be found at 
https://pip.pypa.io/en/latest/development/release-process/#python-2-support
   Package                  Version
   ------------------------ ---------
   apache-bookkeeper-client 4.9.2
   certifi                  2019.9.11
   chardet                  3.0.4
   enum34                   1.1.6
   fastavro                 0.22.5
   futures                  3.3.0
   grpcio                   1.24.1
   idna                     2.8
   kazoo                    2.6.1
   pip                      19.3.1
   prometheus-client        0.7.1
   protobuf                 3.10.0
   pulsar-client            2.5.0
   pymmh3                   0.0.5
   pytz                     2019.3
   PyYAML                   5.1.2
   ratelimit                2.2.1
   requests                 2.22.0
   setuptools               40.8.0
   six                      1.12.0
   urllib3                  1.25.6
   wheel                    0.33.6
   root@c4b89a6b7021:/pulsar# pip uninstall PyYAML==5.1.2
   Uninstalling PyYAML-5.1.2:
     Would remove:
       /usr/local/lib/python3.7/dist-packages/PyYAML-5.1.2.dist-info/*
       /usr/local/lib/python3.7/dist-packages/yaml/*
   Proceed (y/n)? y
     Successfully uninstalled PyYAML-5.1.2
   root@c4b89a6b7021:/pulsar# pip install PyYAML==5.1.2
   Processing 
/root/.cache/pip/wheels/d9/45/dd/65f0b38450c47cf7e5312883deb97d065e030c5cca0a365030/PyYAML-5.1.2-cp37-cp37m-linux_x86_64.whl
   Installing collected packages: PyYAML
   Successfully installed PyYAML-5.1.2
   ```
   
   After the integration test passes, I will try to upgrade to the new version.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services