Skip to content

Use Python3 in Docker image#13366

Closed
nineinchnick wants to merge 1 commit intotrinodb:masterfrom
nineinchnick:docker-python3
Closed

Use Python3 in Docker image#13366
nineinchnick wants to merge 1 commit intotrinodb:masterfrom
nineinchnick:docker-python3

Conversation

@nineinchnick
Copy link
Copy Markdown
Member

Description

Python 2 has reached EOL and the launcher works fine with Python 3. I also think Trino used Python 3 before changing the base image, so I'd consider this a bugfix.

Is this change a fix, improvement, new feature, refactoring, or other?
fix

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)
docker image

How would you describe this change to a non-technical end user or system administrator?
n/a

Related issues, pull requests, and links

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

( ) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Jul 26, 2022
@nineinchnick
Copy link
Copy Markdown
Member Author

I'm not sure if this requires release notes.

Copy link
Copy Markdown
Member

@mosabua mosabua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call.

@nineinchnick
Copy link
Copy Markdown
Member Author

Not needed anymore since #13376 got merged

@nineinchnick nineinchnick deleted the docker-python3 branch May 27, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants