Make sure don't miss the provider(module)

wu-sheng · wu-sheng · commit 1f4e8717cd2c · 2017-11-30T15:24:38.000+08:00
diff --git a/apm-collector/apm-collector-core/src/main/java/org/skywalking/apm/collector/core/module/BootstrapFlow.java b/apm-collector/apm-collector-core/src/main/java/org/skywalking/apm/collector/core/module/BootstrapFlow.java
@@ -103,10 +103,12 @@ private void makeSequence() throws CycleDependencyException {
                     if (isAllRequiredModuleStarted) {
                         startupSequence.add(provider);
                         allProviders.remove(i);
+                        i--;
                     }
                 } else {
                     startupSequence.add(provider);
                     allProviders.remove(i);
+                    i--;
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -103,10 +103,12 @@ private void makeSequence() throws CycleDependencyException {`
`103`	`103`	`if (isAllRequiredModuleStarted) {`
`104`	`104`	`startupSequence.add(provider);`
`105`	`105`	`allProviders.remove(i);`
	`106`	`+ i--;`
`106`	`107`	`}`
`107`	`108`	`} else {`
`108`	`109`	`startupSequence.add(provider);`
`109`	`110`	`allProviders.remove(i);`
	`111`	`+ i--;`
`110`	`112`	`}`
`111`	`113`	`}`
`112`	`114`