Skip to content

improve: status cache for next reconciliation - only the lock version #2800

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
May 16, 2025
Merged

improve: status cache for next reconciliation - only the lock version #2800

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
8438975
improve: PrimaryUpdateAndCacheUtils naming and javadoc improvements
csviri May 14, 2025
6e5d46e
improve: checks on inputs and lock method version of PrimaryUpdateAnd…
csviri May 14, 2025
1992acd
wip
csviri May 14, 2025
01a9826
wip
csviri May 14, 2025
562416a
wip
csviri May 14, 2025
cfd0ad7
Integration test
csviri May 14, 2025
e966d57
increase default
csviri May 14, 2025
4caf4d0
docs
csviri May 15, 2025
9a2d5a3
improve: status cache for next reconciliation - only the lock version
csviri May 15, 2025
9f27ac9
unit tests
csviri May 15, 2025
16bc52a
fix: typos and wording
metacosm May 16, 2025
f95e732
docs: improve javadoc
metacosm May 16, 2025
9411c65
rename methods
csviri May 16, 2025
dba5df8
naming
csviri May 16, 2025
eac8d3c
fix naming of method
csviri May 16, 2025
9f3e501
docs
csviri May 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
naming 
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
  • Loading branch information
@csviri
csviri committed May 16, 2025
commit dba5df884fe611e76c4a855d19e84d00c91f67b1
4 changes: 2 additions & 2 deletions .../src/main/java/io/javaoperatorsdk/operator/api/reconciler/PrimaryUpdateAndCacheUtils.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public static <P extends HasMetadata> P updateStatusAndCacheResource(
* Patches the status using JSON Merge Patch with optimistic locking and caches the result for
* next reconciliation. For details see {@link #updateAndCacheResource}.
*/
public static <P extends HasMetadata> P jsonMergePatchStatusAndCacheResource(
public static <P extends HasMetadata> P mergePatchStatusAndCacheResource(
P primary, Context<P> context, UnaryOperator<P> modificationFunction) {
return updateAndCacheResource(
primary, context, modificationFunction, r -> context.getClient().resource(r).patchStatus());
Expand All @@ -57,7 +57,7 @@ public static <P extends HasMetadata> P jsonMergePatchStatusAndCacheResource(
* Patches the status using JSON Patch with optimistic locking and caches the result for next
* reconciliation. For details see {@link #updateAndCacheResource}.
*/
public static <P extends HasMetadata> P jsonPatchStatusAndCacheResource(
public static <P extends HasMetadata> P patchStatusAndCacheResource(
P primary, Context<P> context, UnaryOperator<P> modificationFunction) {
return updateAndCacheResource(
primary,
Expand Down
Loading