Skip to content

Commit a6b31ed

Browse files
committed
renames
1 parent 386af60 commit a6b31ed

File tree

2 files changed

+10
-15
lines changed

2 files changed

+10
-15
lines changed

operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventScheduler.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,13 +86,13 @@ void scheduleEvent(CustomResourceEvent event) {
8686
if (eventStore.containsNotScheduledEvent(event.resourceUid())) {
8787
log.debug("Replacing not scheduled event with actual event." +
8888
" New event: {}", event);
89-
eventStore.addOrReplaceEventAsNotScheduledYet(event);
89+
eventStore.addOrReplaceEventAsNotScheduled(event);
9090
return;
9191
}
9292
if (eventStore.containsEventUnderProcessing(event.resourceUid())) {
9393
log.debug("Scheduling event for later processing since there is an event under processing for same kind." +
9494
" New event: {}", event);
95-
eventStore.addOrReplaceEventAsNotScheduledYet(event);
95+
eventStore.addOrReplaceEventAsNotScheduled(event);
9696
return;
9797
}
9898

@@ -118,7 +118,7 @@ void eventProcessingFinishedSuccessfully(CustomResourceEvent event) {
118118
eventStore.removeEventUnderProcessing(event.resourceUid());
119119
if (eventStore.containsNotScheduledEvent(event.resourceUid())) {
120120
log.debug("Scheduling recent event for processing processing: {}", event);
121-
scheduleEvent(eventStore.removeEventNotScheduledYet(event.resourceUid()));
121+
scheduleEvent(eventStore.removeEventNotScheduled(event.resourceUid()));
122122
}
123123
} finally {
124124
lock.unlock();
@@ -130,7 +130,7 @@ void eventProcessingFailed(CustomResourceEvent event) {
130130
lock.lock();
131131
eventStore.removeEventUnderProcessing(event.resourceUid());
132132
if (eventStore.containsNotScheduledEvent(event.resourceUid())) {
133-
CustomResourceEvent notScheduledEvent = eventStore.removeEventNotScheduledYet(event.resourceUid());
133+
CustomResourceEvent notScheduledEvent = eventStore.removeEventNotScheduled(event.resourceUid());
134134
log.debug("Event processing failed. Scheduling the most recent event. Failed event: {}," +
135135
" Most recent event: {}", event, notScheduledEvent);
136136
scheduleEvent(notScheduledEvent);

operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventStore.java

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,23 @@
11
package com.github.containersolutions.operator.processing;
22

3-
import org.slf4j.Logger;
4-
import org.slf4j.LoggerFactory;
5-
63
import java.util.HashMap;
74
import java.util.Map;
85

96
public class EventStore {
107

11-
private final static Logger log = LoggerFactory.getLogger(EventStore.class);
12-
13-
private final Map<String, CustomResourceEvent> eventsNotScheduledYet = new HashMap<>();
8+
private final Map<String, CustomResourceEvent> eventsNotScheduled = new HashMap<>();
149
private final Map<String, CustomResourceEvent> eventsUnderProcessing = new HashMap<>();
1510

1611
public boolean containsNotScheduledEvent(String uuid) {
17-
return eventsNotScheduledYet.containsKey(uuid);
12+
return eventsNotScheduled.containsKey(uuid);
1813
}
1914

20-
public CustomResourceEvent removeEventNotScheduledYet(String uid) {
21-
return eventsNotScheduledYet.remove(uid);
15+
public CustomResourceEvent removeEventNotScheduled(String uid) {
16+
return eventsNotScheduled.remove(uid);
2217
}
2318

24-
public void addOrReplaceEventAsNotScheduledYet(CustomResourceEvent event) {
25-
eventsNotScheduledYet.put(event.resourceUid(), event);
19+
public void addOrReplaceEventAsNotScheduled(CustomResourceEvent event) {
20+
eventsNotScheduled.put(event.resourceUid(), event);
2621
}
2722

2823
public boolean containsEventUnderProcessing(String uuid) {

0 commit comments

Comments
 (0)