-1. (non-binding) These Dockerfiles are very bloated and imo only useful for creating a build environment or running tests. Just as you wouldn't setup a server for a service and then install 200 packages that may or may not be used for the service I wouldn't recommend using these Dockerfiles at runtime. Runtime Dockerfiles should in my opinion be as lightweight and suited to their task as possible.
On Wed, Oct 17, 2018, 1:58 AM Hagay Lupesko <lupe...@gmail.com> wrote: > The PR provides a good explanation of this change and all code updates. > LGTM. > > On Tue, Oct 16, 2018 at 8:41 AM Pedro Larroy <pedro.larroy.li...@gmail.com > > > wrote: > > > Hi > > > > I would like to rename the dockerfiles since they are used as a runtime > > environment and not only as build as they were initially intended. > > > > More info about the change in this PR: > > https://github.com/apache/incubator-mxnet/pull/12423/files > > > > > > Pedro. > > >