[GitHub] [systemds] sebwrede commented on pull request #1721: [SYSTEMDS-3280] Docker Setup For HE

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

sebwrede commented on PR #1721:
URL: https://github.com/apache/systemds/pull/1721#issuecomment-1303080309

   > Can we perhaps make a different docker file that contains the homomorphic encryption and not the R libs maybe, and then build inside this GitHub action?
   
   How can we make sure that we don't need the R libs for the HE tests? Currently it may be true, but what if we add some other tests later on where both HE and R libs are needed?
   Additionally, I think it would be a bit confusing if you need two different Docker images to run all tests instead of having a single Docker image to run all tests. 
   Or did you mean that we should then include the HE dockerfile in the existing test dockerfile so that we still build a single image, but just have the configurations in two separate files? 


-- 
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-unsubscribe@systemds.apache.org

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