-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[type:feat] upgrade dockerfile java runtime version 8 to 17 #5374
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5374 +/- ##
=========================================
Coverage 61.49% 61.50%
- Complexity 8818 8820 +2
=========================================
Files 1261 1261
Lines 39273 39284 +11
Branches 3725 3726 +1
=========================================
+ Hits 24151 24161 +10
- Misses 12952 12954 +2
+ Partials 2170 2169 -1 ☔ View full report in Codecov by Sentry. |
… dockerfile-java-version-17
* Tests * Updated lib.rs and added a README.md I updated the `lib.rs` and updated the WASM file and also added a README.md * Remove TestWasmPluginDiscoveryHandler#handlerDiscoveryUpstreamData * Revert "Remove TestWasmPluginDiscoveryHandler#handlerDiscoveryUpstreamData" This reverts commit c85d3db. * Update AbstractWasmDiscoveryHandlerTest.java --------- Co-authored-by: loongs-zhang <zhangzicheng@apache.org>
* fix AbstractWasmPluginDataHandlerTest * [ISSUE apache#5427] add AbstractWasmMetaDataHandlerTest
…he#5463) Co-authored-by: likeguo <33576070+li-keguo@users.noreply.github.com>
…apache#5462) * add offline in grpc and websocket * trigger ci * trigger ci --------- Co-authored-by: xiaoyu <xiaoyu@apache.org> Co-authored-by: loongs-zhang <zhangzicheng@apache.org>
Co-authored-by: moremind <hefengen@apache.org>
… dockerfile-java-version-17
@tomsun28 this job is done |
LGTM |
openjdk:17-jdk-alpine
Do not merge now, please consider merge after v2.6.1 version realease.
Make sure that:
./mvnw clean install -Dmaven.javadoc.skip=true
.