csviri commented on code in PR #496: URL: https://github.com/apache/spark-kubernetes-operator/pull/496#discussion_r2797910646
########## spark-operator/src/main/java/org/apache/spark/k8s/operator/utils/StatusRecorder.java: ########## Review Comment: Hi just a question so I understand this, why is the resource locked in this case, it seems to be if some changes the resource meanwhile, this will awlays retry up to `maxRetry` number. In this case would be benefital to re-read the resource apply the changes again and do the update. Or just don't use Optimistic locking. In JOSDK 5.3 there will be nice utilities to make it much smarter, will come back to you if it is released. -- 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]
