pegasas commented on code in PR #15303:
URL: 
https://github.com/apache/dolphinscheduler/pull/15303#discussion_r1423576620


##########
.github/workflows/e2e-k8s.yml:
##########
@@ -47,11 +47,19 @@ jobs:
     needs: paths-filter
     if: ${{ (needs.paths-filter.outputs.not-ignore == 'true') || 
(github.event_name == 'push') }}
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        java: [ '8', '11' ]

Review Comment:
   It seems we directly use mvnw for building image, which may not support 
JDK11.
   I found that there is docker CLI in below steps.
   How about we use mvnw for building jar, use Dockerfile for building image?
   Is there any policy in apache against base docker image? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to