[GitHub] [systemds] Baunsgaard commented on pull request #1271: [WIP][SYSTEMDS-2974][Design Doc] Use SystemDS with Docker

[GitBox <gi...@apache.org>]

Baunsgaard commented on pull request #1271:
URL: https://github.com/apache/systemds/pull/1271#issuecomment-836484070


   :-1: for now.
   
   I like the idea of separating the docker images in accordance to what you propose because it fits well with other systems separation of images.
   
   I mainly say :-1: because this is done in the wrong order, especially because the design docs are describing something we already have. I would prefer such a thing as this to be on the dev list, as a suggestion that could be discussed, if it is only documentation, and not an implementation modifying the system. if we want to change the implementation to fit to these docs we should do it in the same process as we update the docker images.


-- 
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:
users@infra.apache.org