parshimers commented on PR #8572: URL: https://github.com/apache/texera/pull/8572#issuecomment-5798425540
Thanks for this, I think it's a great and relatively lightweight addition. I would like to see the ability to separately specify the runtime class for the CU. There is a clear difference in security domain between what runs on the CU and what runs in other services. The CU itself contains untrusted user code, whereas the other services may not. You still might want to use a different runtime for these services, but it's not the same potential threat. I think making the flag use one runtime class for everything is unnecessarily conflating the two things. -- 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]
