improve: status cache for next reconciliation - only the lock version#2800
improve: status cache for next reconciliation - only the lock version#2800
Conversation
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
…CacheUtils Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
|
Alternative to: #2799 |
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Chris Laprun <claprun@redhat.com>
Signed-off-by: Chris Laprun <claprun@redhat.com>
| .getControllerEventSource() | ||
| .handleRecentResourceUpdate(ResourceID.fromResource(primary), updatedResource, primary); | ||
| return updatedResource; | ||
| public static <P extends HasMetadata> P editStatusAndCacheResource( |
There was a problem hiding this comment.
I think this method should be renamed patchStatusAndCacheResource to make it clearer that it's using a patch operation. I understand the idea to follow the fabric8 naming but in this case, it's just a bad name :)
There was a problem hiding this comment.
Yeah, problem is that already there is a patchStatusAndCacheResource., but maybe jsonPatchStatusAndCacheResource ?
There was a problem hiding this comment.
ok removed the json prefixes, seemed to long
| logWarnIfResourceVersionPresent(primary); | ||
| return patchAndCacheStatus( | ||
| primary, context, () -> context.getClient().resource(primary).editStatus(operation)); | ||
| public static <P extends HasMetadata> P patchStatusAndCacheResource( |
There was a problem hiding this comment.
This method should be renamed mergePatchStatusAndCacheResource, imo.
There was a problem hiding this comment.
maybe jsonMergePatchStatusAndCacheResource?
There was a problem hiding this comment.
ok removed the json prefixes, seemed to long
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
No description provided.