aljoscha commented on pull request #14096:
URL: https://github.com/apache/flink/pull/14096#issuecomment-729021607


   They are different in what configuration they allow from the outside, so 
things that testcontainers expects to work don't work for some reason. For 
reference, these are the ES images from the ES docker repo: 
https://www.docker.elastic.co/r/elasticsearch. But the `5.3.3` image from there 
also doesn't work out-of-box with what I'm trying to do. I suggest to merge 
this as is once CI is green and someone else can try and make it work for ES5 
if they want to. But I think it's not worth the effort since I'm hoping we can 
deprecate/remove ES5 soon.


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