Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump Flink 1.17 to 1.17.1 #4877

Closed
wants to merge 6 commits into from
Closed

Conversation

link3280
Copy link
Contributor

Why are the changes needed?

As titled.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before make a pull request

@link3280 link3280 changed the title Bump Flink 1.17.1-rc1 [WIP] Bump Flink 1.17.1-rc1 May 23, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 23, 2023

Codecov Report

Merging #4877 (bfd865a) into master (aee9b94) will not change coverage.
The diff coverage is n/a.

❗ Current head bfd865a differs from pull request most recent head c94c005. Consider uploading reports for the commit c94c005 to get more accurate results

@@          Coverage Diff           @@
##           master   #4877   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         556     556           
  Lines       30697   30677   -20     
  Branches     4000    3996    -4     
======================================
+ Misses      30697   30677   -20     

see 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pan3793
Copy link
Member

pan3793 commented May 23, 2023

Nice work

@bowenliang123 bowenliang123 changed the title [WIP] Bump Flink 1.17.1-rc1 [WIP] Bump Flink 1.17 to 1.17.1-rc1 May 25, 2023
@link3280 link3280 changed the title [WIP] Bump Flink 1.17 to 1.17.1-rc1 Bump Flink 1.17 to 1.17.1 May 26, 2023
@pan3793 pan3793 added this to the v1.8.0 milestone May 27, 2023
@pan3793
Copy link
Member

pan3793 commented May 27, 2023

Thanks, merged to master

@pan3793 pan3793 closed this in ff8611e May 27, 2023
huangzhir pushed a commit to huangzhir/kyuubi that referenced this pull request May 30, 2023
### _Why are the changes needed?_

As titled.

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request

Closes apache#4877 from link3280/bump-flink-1.17.1.

Closes apache#4877

c94c005 [Paul Lin] Bump Flink 1.17.1
bfd865a [Paul Lin] Fix invalid download url
f2b0955 [Paul Lin] Fix flink jar not found with no profile specified
f63f384 [Paul Lin] Fix flink download error with no profile specified
3055b6c [Paul Lin] Fix spotless
07b62fd [Paul Lin] Bump Flink 1.17.1-rc1

Authored-by: Paul Lin <paullin3280@gmail.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants