flo-l commented on a change in pull request #1525:
URL: https://github.com/apache/systemds/pull/1525#discussion_r793761220



##########
File path: 
src/main/java/org/apache/sysds/runtime/controlprogram/federated/FederatedWorkerHandler.java
##########
@@ -417,9 +417,11 @@ private FederatedResponse execUDF(FederatedRequest 
request, ExecutionContextMap
                ExecutionContext ec = ecm.get(request.getTID());
 
                // get function and input parameters
+               // FIXME: handle HE privacy constraints

Review comment:
       haha thank you for this extensive write-up! OK this clears things up for 
me. I've already implemented the automatic switch to homomorphic encryption. As 
I'm only accessing the features and labels in one UDF, I will just load these 
without going through the PrivacyMonitor. I've made the relevant changes in 
commit 9ac7f62d1088ff43827d8fa1b5a056d88061fe86.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to