roadan commented on a change in pull request #56: Ama cli docker URL: https://github.com/apache/incubator-amaterasu/pull/56#discussion_r295650315
########## File path: ama-cli/src/main/kotlin/org/apache/amaterasu/ama/cli/container/ContainerHandler.kt ########## @@ -0,0 +1,67 @@ +package org.apache.amaterasu.ama.cli.container + +import com.github.dockerjava.api.DockerClient +import com.github.dockerjava.api.model.BuildResponseItem +import com.github.dockerjava.core.DefaultDockerClientConfig +import com.github.dockerjava.core.DockerClientBuilder +import com.github.dockerjava.core.command.BuildImageResultCallback +import com.github.dockerjava.jaxrs.JerseyDockerCmdExecFactory +import org.slf4j.LoggerFactory +import java.io.File + + +class ContainerHandler(val action : String) { + val logger = LoggerFactory.getLogger(ContainerHandler::class.java) Review comment: You can simply derive from KLogging ---------------------------------------------------------------- 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 With regards, Apache Git Services