Skip to content

Conversation

@1996fanrui
Copy link
Member

@1996fanrui 1996fanrui commented Feb 13, 2025

What is the purpose of the change

1.20.1 is available in mvn repo. It fixed several of bugs, and we always use the .1 (patch version) as the dependency to avoid some potential bugs.

Brief change log

[FLINK-37236] Bump the flink version from 1.20.0 to 1.20.1

@gyfora
Copy link
Contributor

gyfora commented Feb 20, 2025

please rebase on main for docker fix

@1996fanrui 1996fanrui marked this pull request as ready for review February 21, 2025 02:49
Copy link
Member Author

@1996fanrui 1996fanrui left a comment

Choose a reason for hiding this comment

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

please rebase on main for docker fix

Thanks @gyfora for the fix, rebased.

@gyfora
Copy link
Contributor

gyfora commented Feb 21, 2025

No changes to the NOTICE file necessary?

@1996fanrui
Copy link
Member Author

@gyfora First of all, thank you for providing the command to generate the dependency list.

No changes to the NOTICE file necessary?

Yes, I tried to run mvn clean package -DskipTests | grep Including | sort | cut -f 3 -d ' ' | sed -e 's/^/- /' in flink-kubernetes-operator and flink-autoscaler-standalone submodule, both of them are not changed.

Also, these 2 dependencies under the BSD License, I corrected them as well.

- com.esotericsoftware.kryo:kryo:jar:2.24.0
- com.esotericsoftware.minlog:minlog:jar:1.2

@gyfora gyfora merged commit d54c8df into apache:main Feb 21, 2025
115 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants