tanishqgandhi1908 commented on code in PR #6869:
URL: https://github.com/apache/texera/pull/6869#discussion_r3827810328


##########
file-service/src/main/scala/org/apache/texera/service/FileService.scala:
##########
@@ -90,6 +92,8 @@ class FileService extends 
Application[FileServiceConfiguration] with LazyLogging
 
     environment.jersey.register(classOf[DatasetResource])
     environment.jersey.register(classOf[DatasetAccessResource])
+    environment.jersey.register(classOf[ModelResource])

Review Comment:
   Both fixed. Added ModelResource and ModelAccessResource to 
FileServiceRunSpec's Seq, and added ModelResourcePermissionsSpec mirroring the 
dataset one — every endpoint carries exactly one of @PermitAll/@RolesAllowed, 
non-public ones require REGULAR/ADMIN, and getPublicModel is pinned as the only 
@PermitAll.
   



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