jiangzho commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2033297657
Thanks for quick turn around @mridulm ! I understand the concern regarding its size. This commit is result of effort to break down the original scope as much as possible. Given that this is a new project, this PR includes several fundamental modules that are essential to make the operator functional. I'd also like to highlight that ~7k lines are auto-generated CRD yaml, which can be bypassed during reviewing its POJO (1k lines). There are ~3k lines comprises of documentations and examples which may require less scrutiny. I appreciate your understanding and open to the discussing how we can streamline the review process together. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
