Hi Mandy, On Fri, 2019-07-19 at 10:32 -0700, Mandy Chung wrote: > I still think separating container-specific APIs in its own class will > prevent running similar kind of issue in the future.
Take 03: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8228434/03/webrev/ This now reverts Platform.java changes to the version prior JDK- 8227642, introduces a Container class with one constant, CONTAINER_ENGINE, and renames the jdk.test.docker.command property to jdk.test.container.command. Breakage as seen via jdk/net/Sockets/Test.java no longer exists as the Platform class no longer loads an extra property. Testing: jdk/net/Sockets/Test.java test, container tests with docker/podman on Linux x86_64. Thoughts? Thanks, Severin
