Re: [PR] [SYSTEMDS-3877] Docker image size reduction and safety improvement [systemds]

[via GitHub]

Baunsgaard commented on PR #2274:
URL: https://github.com/apache/systemds/pull/2274#issuecomment-3018145898

   > ## Size Reduction
   > ### Before
   > 
   > 
![Before](https://private-user-images.githubusercontent.com/51443671/458954908-eeff534c-4610-405f-a95a-a436b450c6b0.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTEyNjk4MzEsIm5iZiI6MTc1MTI2OTUzMSwicGF0aCI6Ii81MTQ0MzY3MS80NTg5NTQ5MDgtZWVmZjUzNGMtNDYxMC00MDVmLWE5NWEtYTQzNmI0NTBjNmIwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTA2MzAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwNjMwVDA3NDUzMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY5OGM2NDExN2Y4ZjUxZjhlNDFlYzlkZmE1M2JiYTBmYmQ2NWNjYTM4NTk1ODkyNzk5NzdhYjgyNWI4OWZlYmQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0._JpJFoxzNcdKVlTqXsX2OVcww3faZIjoVRhyyHijgrg)
   > ### After
   > 
   > 
![After](https://private-user-images.githubusercontent.com/51443671/458954904-927920aa-a724-4a40-b58a-b1b5c38d2200.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NTEyNjk4MzEsIm5iZiI6MTc1MTI2OTUzMSwicGF0aCI6Ii81MTQ0MzY3MS80NTg5NTQ5MDQtOTI3OTIwYWEtYTcyNC00YTQwLWI1OGEtYjFiNWMzOGQyMjAwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTA2MzAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwNjMwVDA3NDUzMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkwYzYzYTc2YzZjNGZkNjVkMzI0NzVmOGVhZjI3MGEzNmFkYmI3NWY2OWEwMWZmODY0NjUxZmM0NTA5ZjkwOWImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.ZjATaHi_LNbPYPG32qg5zyiRMS2nzhPislPYd2-uLYo)
   > 
   > We reduced the size of the main docker image by 1.11 GB.
   
   Hi @alexanderschmi 
   
   Thanks for the PR, it is a great improvement!
   
   Based on your pictures, did we move the SystemDS folder to the root? If so, 
would it not be better if we move it back to the previous position?
   
   Best regards
   Sebastian


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

To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org