Skip to content

Commit 70834e3

Browse files
authored
[ISSUE #10009] Convert internal-only public classes to inner classes and add null-check log (#10028)
Change-Id: I9cc200303d3278c2cf763b953dd1ccafff57bcd2
1 parent 4a29b7e commit 70834e3

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

broker/src/main/java/org/apache/rocketmq/broker/schedule/ScheduleMessageService.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -546,7 +546,7 @@ private PutResultProcess deliverMessage(MessageExtBrokerInner msgInner, String m
546546
}
547547
}
548548

549-
public class HandlePutResultTask implements Runnable {
549+
class HandlePutResultTask implements Runnable {
550550
private final int delayLevel;
551551

552552
public HandlePutResultTask(int delayLevel) {
@@ -558,6 +558,12 @@ public void run() {
558558
LinkedBlockingQueue<PutResultProcess> pendingQueue =
559559
ScheduleMessageService.this.deliverPendingTable.get(this.delayLevel);
560560

561+
// Check if the queue exists for the given level
562+
if (pendingQueue == null) {
563+
log.warn("No pending queue found for delay level: {}", this.delayLevel);
564+
return;
565+
}
566+
561567
PutResultProcess putResultProcess;
562568
while ((putResultProcess = pendingQueue.peek()) != null) {
563569
try {
@@ -599,7 +605,7 @@ private void scheduleNextTask() {
599605
}
600606
}
601607

602-
public class PutResultProcess {
608+
class PutResultProcess {
603609
private String topic;
604610
private long offset;
605611
private long physicOffset;
@@ -824,7 +830,7 @@ public String toString() {
824830
}
825831
}
826832

827-
public enum ProcessStatus {
833+
enum ProcessStatus {
828834
/**
829835
* In process, the processing result has not yet been returned.
830836
*/

0 commit comments

Comments
 (0)