Uploaded image for project: 'Flink'
  1. Flink
  2. FLINK-34195

PythonEnvUtils creates python environment instead of python3

    XMLWordPrintableJSON

Details

    Description

      I was looking into the Python installation of the Flink test suite because I working on updating the CI Docker image from 16.04 (Xenial) to 22.04 (FLINK-34194). I noticed that there is test code still relying on the python command instead of python3. For Ubuntu 16.04 that meant relying on Python 2. Therefore, we have tests still relying on Python 2 as far as I understand.

      I couldn't find any documentation or mailing list discussion on major Python version support. But AFAIU, we're relying on Python3 (based on the e2e tests) which makes these tests out-dated.

      Additionally, python.client.executable relies on python.

      Should we make it more explicit in our test code that we're actually expecting python3? Additionally, should that be mentioned somewhere in the docs? Or if it's already mentioned, could you point me to it? (As someone looking into PyFlink for the "first" time) I would have expected something like that being mentioned on the PyFlink overview. Or is it the default to assume nowadays that python refers to python3?

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              mapohl Matthias Pohl
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated: