[BugFix] Fix getNextWorker overflow (backport #53213)

[mergify]

Why I'm doing:

The getNextWorker() method uses a global AtomicInteger nextIndexId, which increments by one with each call to getNextWorker(). As a result, it may overflow into a negative value.

java.lang.ArrayIndexOutOfBoundsException: Index -1 out of bounds for length 3
        at com.google.common.collect.RegularImmutableMap$Values.get(RegularImmutableMap.java:384) ~[spark-dpp-1.0.0.jar:?]
        at com.starrocks.qe.scheduler.DefaultWorkerProvider.getNextWorker(DefaultWorkerProvider.java:354) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.DefaultWorkerProvider.selectNextWorker(DefaultWorkerProvider.java:141) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.assignment.RemoteFragmentAssignmentStrategy.assignGatherFragmentToWorker(RemoteFragmentAssignmentStrategy.java:91) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.assignment.RemoteFragmentAssignmentStrategy.assignFragmentToWorker(RemoteFragmentAssignmentStrategy.java:76) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.CoordinatorPreprocessor.computeFragmentInstances(CoordinatorPreprocessor.java:244) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.CoordinatorPreprocessor.prepareExec(CoordinatorPreprocessor.java:200) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.DefaultCoordinator.prepareExec(DefaultCoordinator.java:456) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.DefaultCoordinator.startScheduling(DefaultCoordinator.java:501) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.Coordinator.startScheduling(Coordinator.java:105) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.Coordinator.exec(Coordinator.java:88) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.StmtExecutor.handleQueryStmt(StmtExecutor.java:1119) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.StmtExecutor.execute(StmtExecutor.java:633) ~[starrocks-fe.jar:?]

What I'm doing:

What type of PR is this:

• BugFix
• Feature
• Enhancement
• Refactor
• UT
• Doc
• Tool

Does this PR entail a change in behavior?

• Yes, this PR will result in a change in behavior.
• No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

• Interface/UI changes: syntax, type conversion, expression evaluation, display information
• Parameter changes: default values, similar parameters but with different default values
• Policy changes: use new policy to replace old one, functionality automatically enabled
• Feature removed
• Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

• I have added test cases for my bug fix or my new feature
• This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function
• This is a backport pr

Bugfix cherry-pick branch check:

• I have checked the version labels which the pr will be auto-backported to the target branch
  • 3.4
  • 3.3
  • 3.2
  • 3.1
  • 3.0

    ---

    This is an automatic backport of pull request [BugFix] Fix getNextWorker overflow #53213 done by Mergify.

Why I'm doing:

The getNextWorker() method uses a global AtomicInteger nextIndexId, which increments by one with each call to getNextWorker(). As a result, it may overflow into a negative value.

java.lang.ArrayIndexOutOfBoundsException: Index -1 out of bounds for length 3
        at com.google.common.collect.RegularImmutableMap$Values.get(RegularImmutableMap.java:384) ~[spark-dpp-1.0.0.jar:?]
        at com.starrocks.qe.scheduler.DefaultWorkerProvider.getNextWorker(DefaultWorkerProvider.java:354) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.DefaultWorkerProvider.selectNextWorker(DefaultWorkerProvider.java:141) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.assignment.RemoteFragmentAssignmentStrategy.assignGatherFragmentToWorker(RemoteFragmentAssignmentStrategy.java:91) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.assignment.RemoteFragmentAssignmentStrategy.assignFragmentToWorker(RemoteFragmentAssignmentStrategy.java:76) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.CoordinatorPreprocessor.computeFragmentInstances(CoordinatorPreprocessor.java:244) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.CoordinatorPreprocessor.prepareExec(CoordinatorPreprocessor.java:200) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.DefaultCoordinator.prepareExec(DefaultCoordinator.java:456) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.DefaultCoordinator.startScheduling(DefaultCoordinator.java:501) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.Coordinator.startScheduling(Coordinator.java:105) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.scheduler.Coordinator.exec(Coordinator.java:88) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.StmtExecutor.handleQueryStmt(StmtExecutor.java:1119) ~[starrocks-fe.jar:?]
        at com.starrocks.qe.StmtExecutor.execute(StmtExecutor.java:633) ~[starrocks-fe.jar:?]

What I'm doing:

What type of PR is this:

• BugFix
• Feature
• Enhancement
• Refactor
• UT
• Doc
• Tool

Does this PR entail a change in behavior?

• Yes, this PR will result in a change in behavior.
• No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

• Interface/UI changes: syntax, type conversion, expression evaluation, display information
• Parameter changes: default values, similar parameters but with different default values
• Policy changes: use new policy to replace old one, functionality automatically enabled
• Feature removed
• Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

• I have added test cases for my bug fix or my new feature
• This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function
• This is a backport pr

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud