Skip to content

[KYUUBI #3573] Bump scala from 2.12.16 to 2.12.17 #3574

Closed
yikf wants to merge 1 commit intoapache:masterfrom
yikf:bump-scala
Closed

[KYUUBI #3573] Bump scala from 2.12.16 to 2.12.17 #3574
yikf wants to merge 1 commit intoapache:masterfrom
yikf:bump-scala

Conversation

@yikf
Copy link
Contributor

@yikf yikf commented Sep 28, 2022

Why are the changes needed?

Close #3573.

This PR aims to upgrade scala to 2.12.17

  1. Scala 2.12.17 is available currently, This version has notable changes affecting compatibility, for more details see release note.
  2. Spark was upgraded to this version half a month ago.

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

@codecov-commenter
Copy link

Codecov Report

Merging #3574 (51521d8) into master (8419b7b) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3574      +/-   ##
============================================
- Coverage     51.74%   51.73%   -0.02%     
  Complexity       13       13              
============================================
  Files           482      482              
  Lines         26975    26975              
  Branches       3762     3762              
============================================
- Hits          13959    13956       -3     
  Misses        11661    11661              
- Partials       1355     1358       +3     
Impacted Files Coverage Δ
...pache/kyuubi/engine/YarnApplicationOperation.scala 62.96% <0.00%> (-1.86%) ⬇️
...he/kyuubi/ha/client/etcd/EtcdDiscoveryClient.scala 66.25% <0.00%> (-0.63%) ⬇️
...a/org/apache/kyuubi/service/TFrontendService.scala 91.17% <0.00%> (-0.30%) ⬇️

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

@pan3793 pan3793 added this to the v1.7.0 milestone Sep 29, 2022
@pan3793
Copy link
Member

pan3793 commented Sep 29, 2022

Thanks, merging to master

@pan3793 pan3793 closed this in 98ff122 Sep 29, 2022
@yikf yikf deleted the bump-scala branch September 29, 2022 02:21
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.

[Improvement] Bump scala from 2.12.16 to 2.12.17

5 participants