jiangzho commented on PR #241: URL: https://github.com/apache/spark-kubernetes-operator/pull/241#issuecomment-2981324852
Thanks for the feedback Dongjoon! I'm thinking that the POJOs may be needed not only for the CRD generation purpose (though this PR only targets that). Without these, our operator seems only listing / watching on v1beta1 despite the chart installs both v1alpha1 and v1beta1. If there's any alpha CRDs alive, the operator won't be reconciling or cleaning them up. For alpha version it could be okay, but we may need a strategy for upgrading versions FMPOV. With a POJO for the old version, it becomes possible to enable additional lister watcher (could be protected by a config param), and reconcile them (as they can be converted to latest using the `toLatestSparkApplication), and users get the transition time to migrate workload to the next version with operator updated. -- 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]
