wangyang0918 commented on pull request #12215:
URL: https://github.com/apache/flink/pull/12215#issuecomment-631977115


   @zhengcanbin Thanks for the updating. LGTM now.
   
   One more thing, we need to revert this commit 
5846e5458e20308db4f0dd1c87d511a60817472c to verify FLINK-17416 is fixed(i.e. 
Flink on K8s could work on java 8u252). For FLINK-17566, i have verified it 
could be fixed by this PR.
   
   nit: Maybe we could use fabric8 kubernetes-client version in the commit 
message, not fabric8 version.
   
   
   Moreover, could you rebase the latest master to enable the e2e test?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to